Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix COUNT(*) ERROR for PDF SELECT Queries #1376

Merged
merged 5 commits into from
Nov 22, 2023
Merged

Fix COUNT(*) ERROR for PDF SELECT Queries #1376

merged 5 commits into from
Nov 22, 2023

Commits on Nov 20, 2023

  1. fix lark_visitor's aggregate function to generate the correct TupleVa… 

    …lueExpression for count(*) groupby
    @jkim3663
    jkim3663 committed Nov 20, 2023
    Configuration menu
    Copy the full SHA
    a70eddd View commit details
    Browse the repository at this point in the history

  2. add sample pdf data for count(*) and groupby testing

    @jkim3663
    jkim3663 committed Nov 20, 2023
    Configuration menu
    Copy the full SHA
    c1065bd View commit details
    Browse the repository at this point in the history

  3. added integration test case for count(*) behavioral

    @jkim3663
    jkim3663 committed Nov 20, 2023
    Configuration menu
    Copy the full SHA
    290ab4e View commit details
    Browse the repository at this point in the history

  4. removed scribble comment; drop table after test case run

    @jkim3663
    jkim3663 committed Nov 20, 2023
    Configuration menu
    Copy the full SHA
    3df7fc6 View commit details
    Browse the repository at this point in the history

  5. formatting

    @jkim3663
    jkim3663 committed Nov 20, 2023
    Configuration menu
    Copy the full SHA
    d505f48 View commit details
    Browse the repository at this point in the history