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

Allow multiple aggregated columns per query #504

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Allow multiple aggregated columns per query #504

wants to merge 10 commits into from

Commits on Jun 25, 2021

  1. Implement column-as keyword for 'select' with SelectAs().

    @dgeelen-uipath
    dgeelen-uipath committed Jun 25, 2021
    Configuration menu
    Copy the full SHA
    4c1446a View commit details
    Browse the repository at this point in the history

  2. Implement column-as keyword for 'count' aggregate as CountAs().

    @dgeelen-uipath
    dgeelen-uipath committed Jun 25, 2021
    Configuration menu
    Copy the full SHA
    0ec80b1 View commit details
    Browse the repository at this point in the history

  3. Replace AsCount() by CountAs().

    @dgeelen-uipath
    dgeelen-uipath committed Jun 25, 2021
    Configuration menu
    Copy the full SHA
    10ae29c View commit details
    Browse the repository at this point in the history

  4. Make aggregate check input columns.

    @dgeelen-uipath
    dgeelen-uipath committed Jun 25, 2021
    Configuration menu
    Copy the full SHA
    ae2bcb2 View commit details
    Browse the repository at this point in the history

  5. Implement as-keyword support for all aggregates.

    @dgeelen-uipath
    dgeelen-uipath committed Jun 25, 2021
    Configuration menu
    Copy the full SHA
    5991a60 View commit details
    Browse the repository at this point in the history

  6. Rename AggregateAs() -> SelectAggregate().

    @dgeelen-uipath
    dgeelen-uipath committed Jun 25, 2021
    Configuration menu
    Copy the full SHA
    2e2e05a View commit details
    Browse the repository at this point in the history

  7. Don't silently drop aggregate components.

    @dgeelen-uipath
    dgeelen-uipath committed Jun 25, 2021
    Configuration menu
    Copy the full SHA
    f34c646 View commit details
    Browse the repository at this point in the history

  8. Restrict aggregate transform to sub-query to COUNT() only: 

    This transform doesn't seem to make sense for aggregates other than count,
as that is the only one that supports a '*' clause.
    @dgeelen-uipath
    dgeelen-uipath committed Jun 25, 2021
    Configuration menu
    Copy the full SHA
    d3e3d57 View commit details
    Browse the repository at this point in the history

  9. Implement allowing multiple aggregated columns per query.

    @dgeelen-uipath
    dgeelen-uipath committed Jun 25, 2021
    Configuration menu
    Copy the full SHA
    0a5610b View commit details
    Browse the repository at this point in the history

  10. Implement tests for aggregates combining with GROUP BY, ORDER BY, and… 

    … LIMIT keywords.
    @dgeelen-uipath
    dgeelen-uipath committed Jun 25, 2021
    Configuration menu
    Copy the full SHA
    e90f364 View commit details
    Browse the repository at this point in the history