rluvaton opened a new pull request, #14299: URL: https://github.com/apache/datafusion/pull/14299
## Which issue does this PR close? N/A ## Rationale for this change We want faster DataFusion ## What changes are included in this PR? Added fast path for `array_agg` accumulator for `merge_batch` that return the underlying values of the list in case there is no null that point to the non empty list starting from the first valid index and until last valid index ## Are these changes tested? Existing tests ## Are there any user-facing changes? Nope ------- I see that there is no benchmark for `array_agg`, so I will try to add one in another PR so we can verify that this is indeed improving performance -- 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
