mzabaluev opened a new pull request, #23648: URL: https://github.com/apache/datafusion/pull/23648
## Which issue does this PR close? - Part of #22682. ## Rationale for this change This adds column-native `GroupColumn` implementations for `List<T>` and `LargeList<T>`, so a `GROUP BY` containing these (along with other supported data types) no longer falls back from `GroupValuesColumn` to `GroupValuesRows`. ## What changes are included in this PR? Cherry-picked @zhuqi-lucas's changes in #22706 that pertained to `List` / `LargeList`, and made some optimizations. ## Are these changes tested? Cherry-picked the unit tests for the `list` module from #22706. Added supported/unsupported cases for `List` in the `group_column_supported_type_matches_make_group_column` test. Benchmarked in a proprietary application using several HashAggregate operations on 20 columns including two string lists, to about 8% cumulative improvement. ## Are there any user-facing changes? No. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
