kumarUjjawal opened a new pull request, #25252: URL: https://github.com/apache/datafusion/pull/25252
## Which issue does this PR close? Part of #25047. ## Rationale for this change `ordered_aggregate_spill.slt` can fail when spill-merge buffers leave too little memory for aggregate replay while another partition retains aggregate state in the shared greedy pool. Limit merge fan-in to two to reduce this contention while preserving the 600 KiB memory limit, two partitions, and existing query results. ## What changes are included in this PR? - Set and reset the SQL test's spill merge fan-in. - Add controlled two-partition tests for grouped results, spilling, and reservation cleanup on completion, input error, and cancellation. - Check spilled rows without depending on the formatted byte unit, and remove exact spill-count claims from comments. ## What is the testing strategy for this PR? Local validation: - 320 SQL test executions (16 concurrent copies per run) and 50 repetitions of the focused tests. - Full extended workspace tests, including all 518 SQL test files. - Workspace Clippy with all targets and features, formatting, and submission checks. ## Are there any user-facing changes? No. This changes tests only; production execution and defaults are unchanged. -- 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]
