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.

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

handle the new grouping behavior using references #299

Open
kadinrabo opened this issue Oct 2, 2024 · 0 comments
Open

handle the new grouping behavior using references #299

kadinrabo opened this issue Oct 2, 2024 · 0 comments

Comments

@kadinrabo
Copy link
Contributor

We should update the behavior in substrait-java with the spec update 706 changing the definition of grouping sets in AggregateRel to consist of references into a list of grouping expressions instead of consisting of expressions directly.

This change keeps consumers from having to deduplicate the expressions in the grouping sets in order to execute the query or even derive the output schema. Explained in further detail in #700.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant