ozankabak commented on PR #16196: URL: https://github.com/apache/datafusion/pull/16196#issuecomment-2925042496
> I just updated the PR also support sort exec, also added end to end testing. This is exactly what I want to avoid -- there are too many execs and we shouldn't have a structure where this is repeated everywhere in code. Let's keep trying various ideas for a unified solution in this PR, and if we fail to do it, we will think about merging the AggregateExec-only part to alleviate any downstream aggregation-related concerns. -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
