Kevin-Li-2025 commented on PR #23066: URL: https://github.com/apache/datafusion/pull/23066#issuecomment-4801224448
Addressed the test coverage suggestion in 8315dce6b. I added a focused unit test that builds four SortedSpillFile entries with max_spill_merge_fan_in = 2 and asserts that one merge phase selects/drains only two spill inputs, leaving the remaining two for a later phase.\n\nLocal validation: git diff --check passed. I could not run cargo fmt/test in this sandbox because rustup tries to write under ~/.rustup/update-hashes, which is not writable here, and a temporary minimal toolchain download stalled before completion. -- 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]
