alamb commented on issue #9213: URL: https://github.com/apache/arrow-datafusion/issues/9213#issuecomment-1956034509
As @Lordworms points out, maybe we can try to implement this feature in some way that is general and not special cased in the optimzer -- aka https://github.com/apache/arrow-datafusion/issues/9289 This would look like 1. Making `ARRAY_AGG` an `AggregateUDF` (which probably would mean making a `datafusion-aggregates` crate 2. Adding an API to do the rewrite This would certainly take more work and thus more time than just implementing a special case for the BuiltInAggregateFunction, so I don't think it is necessary However, if we think this is a reasonbale approach I can file some tickets with the basic ideas sketeched out (I didn't want to sketch out too many things at once and we already have a bunch of work related to pulling out scalar UDFs) -- 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]
