2010YOUY01 opened a new issue, #24317: URL: https://github.com/apache/datafusion/issues/24317
### Is your feature request related to a problem or challenge? The spilling path added for `OrderedSingleAggregateStream` in #24259 is largely similar to the spilling paths used by other aggregation modes. Keeping separate copies of the same spill mechanics makes future fixes and changes easier to miss and can cause the implementations to drift. This is follow-up work from https://github.com/apache/datafusion/pull/24259#discussion_r3757233396 and part of #22710. ### Describe the solution you'd like Identify the semantically identical parts of the aggregation spilling paths and extract them into shared logic that can be reused by the relevant aggregation streams. Keep mode-specific policies and state transitions explicit in each dedicated stream; only the common spill mechanics should be shared. ### Describe alternatives you've considered ### Additional context -- 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]
