2010YOUY01 opened a new issue, #23113: URL: https://github.com/apache/datafusion/issues/23113
### Is your feature request related to a problem or challenge? - https://github.com/apache/datafusion/issues/22710 In the above EPIC's aggregate refactor, it's now using a variant of `AggregateHashTable<ParitalSkip>` to handle the `convert_to_states()` optimization (see https://github.com/apache/datafusion/pull/22899 for this optimization) https://github.com/apache/datafusion/blob/main/datafusion/physical-plan/src/aggregates/hash_table.rs#L601 The only needed fields are only `GroupsAccumulators`, and we don't have to carry along the heavy hash table, so we can simplify `AggregateHashTable<PartialSkip>` into a simpler struct. ### Describe the solution you'd like _No response_ ### Describe alternatives you've considered _No response_ ### Additional context _No response_ -- 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]
