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

add CursorHolder.isPreAggregated method to allow cursors on pre-aggregated data #17058

Merged
merged 2 commits into from
Sep 13, 2024
Merged

add CursorHolder.isPreAggregated method to allow cursors on pre-aggregated data #17058

merged 2 commits into from
Sep 13, 2024

Commits on Sep 13, 2024

  1. add CursorHolder.isPreAggregated method to allow cursors on pre-aggre… 

    …gated data

changes:
* CursorHolder.isPreAggregated method indicates that a cursor has pre-aggregated data for all AggregatorFactory specified in a CursorBuildSpec. If true, engines should rewrite the query to use AggregatorFactory.getCombiningAggreggator, and column selector factories will provide selectors with the aggregator interediate type for the aggregator factory name
* Added groupby, timeseries, and topN support for CursorHolder.isPreAggregated
* Added synthetic test since no CursorHolder implementations support isPreAggregated at this point in time
    @clintropolis
    clintropolis committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    f1d92b0 View commit details
    Browse the repository at this point in the history

  2. style

    @clintropolis
    clintropolis committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    7afb120 View commit details
    Browse the repository at this point in the history