Re: [PR] Minor: remove unused IPCWriter [datafusion]

2025-05-31 Thread via GitHub
alamb commented on PR #16215: URL: https://github.com/apache/datafusion/pull/16215#issuecomment-2925498438 Thanks @comphead and @2010YOUY01 -- 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 sp

Re: [PR] Minor: remove unused IPCWriter [datafusion]

2025-05-30 Thread via GitHub
2010YOUY01 merged PR #16215: URL: https://github.com/apache/datafusion/pull/16215 -- 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...@dat

Re: [PR] Minor: remove unused IPCWriter [datafusion]

2025-05-30 Thread via GitHub
alamb commented on PR #16215: URL: https://github.com/apache/datafusion/pull/16215#issuecomment-2922905121 > hope downstream users didn't use it If they need to write data to IPC files, I think either of the upstream arrow-rs choices is better: 1. https://docs.rs/arrow-ipc/55.1.0/a

Re: [PR] Minor: remove unused IPCWriter [datafusion]

2025-05-30 Thread via GitHub
comphead commented on PR #16215: URL: https://github.com/apache/datafusion/pull/16215#issuecomment-2922744744 hope downstream users didn't use it -- 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 t

[PR] Minor: remove unused IPCWriter [datafusion]

2025-05-30 Thread via GitHub
alamb opened a new pull request, #16215: URL: https://github.com/apache/datafusion/pull/16215 ## Which issue does this PR close? ## Rationale for this change DataFusion now uses the [SpillManager](https://github.com/apache/datafusion/blob/b6a51740f1c3cd92e416fea4d2f3b1600f1