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/b6a51740f1c3cd92e416fea4d2f3b1600f10c97a/datafusion/physical-plan/src/spill/spill_manager.rs#L41) code for writing spill files and `IpcWriter` is unused This is also duplicative with code upstream ## What changes are included in this PR? Remove unused code ## Are these changes tested? By CI ## Are there any user-facing changes? This is technically a pub struct so this is an API change. If we wanted to lower the potential impact we could mark the struct as deprecated instsead but given it is old and unused I thought it would be ok to just delete 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 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
