substrait-io / substrait-java

Apache License 2.0
77 stars 72 forks source link

handle the new grouping behavior using references #299

Open kadinrabo opened 1 month ago

kadinrabo commented 1 month ago

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.