substrait-io / substrait-java

Apache License 2.0
77 stars 72 forks source link

fix: propagate sorts in aggregate function invocation proto->rel #313

Closed bvolpato closed 1 month ago

bvolpato commented 1 month ago

When consuming a Substrait proto plan that contains sort within aggregate calls, the SortFields were being dropped in the process.

This is what I mean by sort within aggregates:

SELECT ARRAY_AGG(genre ORDER BY genre) FROM book

So this is a simple addition + test case change that closes that gap.