bvolpato opened a new pull request, #13114: URL: https://github.com/apache/datafusion/pull/13114
## Which issue does this PR close? Closes #13113. ## Rationale for this change Adding support to a new sort of expression (ORDER BY within aggregation functions). For example: > SELECT ARRAY_AGG(genre **ORDER BY genre**) FROM book ## What changes are included in this PR? The wiring to go from Substrait's SortField to DataFusion's SortExpr in the context of aggregation measurements. ## Are these changes tested? I added a roundtrip test and also added a logical plan to check if everything was decoded correctly. Open to more ideas on how to better unit test this. ## Are there any user-facing changes? n/a -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
