2010YOUY01 commented on PR #15405: URL: https://github.com/apache/datafusion/pull/15405#issuecomment-2753544418
> Thanks @2010YOUY01 lgtm I'm also thinking if we should give a spill manager the read spilled files functions? Yes there is already a basic one: https://github.com/apache/datafusion/blob/b2430117f1525d80d4053d7d689b2d74a14d26b8/datafusion/physical-plan/src/spill/spill_manager.rs#L119 In the future if there are more functions to read spilled file I think there should also be included inside `SpillManager` -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
