ozankabak commented on PR #8662: URL: https://github.com/apache/arrow-datafusion/pull/8662#issuecomment-1871436169
> I believe if we applied the same change to ArrayAgg I think we can remove the limitation of a single compatible ORDER BY in a query -- aka https://github.com/apache/arrow-datafusion/issues/8582 -- is that your understanding too? Yes, if we make `ARRAY_AGG` sort internally, we can do this > As we discussed in the design document the potential downside of this approach is that if multiple aggregates share the same ORDER BY clause, they will each independently sort the input batches, which is unfortunate but could be optimized in future PTs Exactly. We will explore both the split/diamond approach and the approach above in the upcoming weeks -- 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]
