datafusion
                                
                                 datafusion copied to clipboard
                                
                                    datafusion copied to clipboard
                            
                            
                            
                        Array agg groups accumulator
Which issue does this PR close?
Closes #10145.
Rationale for this change
See the issue.
What changes are included in this PR?
GroupsAccumulator for array_agg aggregation function for:
- Primitive types
- String type
Not included:
- Accumulating arrays of any level of nesting.
Are these changes tested?
Extended tests in array_agg.rs
Are there any user-facing changes?
No, just a performance improvement.
Thank you for this PR @lkt -- I will take a look at this one carefully shortly
Thanks @lkt for your contribution, does this PR stand for array_agg performance improvement? If so it is good to see a bench mark results
Thank you for your contribution. Unfortunately, this pull request is stale because it has been open 60 days with no activity. Please remove the stale label or comment or this will be closed in 7 days.
Seems to be superceded by https://github.com/apache/datafusion/pull/11096
Thank you for your contribution. Unfortunately, this pull request is stale because it has been open 60 days with no activity. Please remove the stale label or comment or this will be closed in 7 days.