Skip to content

Composite aggregates #3266

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.

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

Open
wants to merge 20 commits into
base: main
Choose a base branch
from

Conversation

normen662
Copy link
Contributor

@normen662 normen662 commented Mar 21, 2025

This PR adds the ability to plan arbitrary aggregation queries using aggregation indexes as well as intersections of aggregation indexes. It also corrects/adds the infrastructure to deal with complicated order-by requirements.

  • refactor data access rules into a data access rule for value indexes and one for aggregation indexes
  • have both of these data access rule implementations share the maximum code (only intersection planning is different)
  • intersection planning for aggregation indexes
    • because of the same reasoning as regular index intersections AND in order to answer the query as these indexes carry data that is not part of the underlying table (the aggregation)
    • rollups of finer-granularity index scans if such a roll-up can be useful in answering the query
  • order by requirements are treated in the same framework as all other kinds of expressions (using M3 maps, translations, etc.)

@normen662 normen662 force-pushed the composite-aggregates branch from a25ae95 to 4c9ec49 Compare March 24, 2025 08:24
@normen662 normen662 force-pushed the composite-aggregates branch 2 times, most recently from 72f7a46 to 2f5e9b8 Compare April 2, 2025 19:26
@normen662 normen662 force-pushed the composite-aggregates branch from 8f97639 to 55cd005 Compare April 10, 2025 12:47
@normen662 normen662 requested review from hatyo, alecgrieser and MMcM April 15, 2025 08:20
@normen662 normen662 added the enhancement New feature or request label Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant