kosiew opened a new pull request, #21917: URL: https://github.com/apache/datafusion/pull/21917
## Which issue does this PR close? * Closes #21914. --- ## Rationale for this change The spill subsystem implicitly depended on `arrow-ipc` codec features (lz4, zstd) via workspace-level feature unification. This created a fragile coupling where changes to workspace features could silently break spill compression at runtime. This PR makes the dependency explicit and local to the crate that owns spill functionality, improving correctness, maintainability, and debuggability. --- ## What changes are included in this PR? * Added an explicit dependency on `arrow-ipc` with `lz4` and `zstd` features in `datafusion-physical-plan/Cargo.toml`. * Documented the codec requirement contract in `IPCStreamWriter`. * Updated `Cargo.lock` to include the new dependency. --- ## Are these changes tested? No new tests are included in this PR. --- ## Are there any user-facing changes? No user-facing changes. This update only affects internal dependency configuration and documentation. --- ## LLM-generated code disclosure This PR includes LLM-generated code and comments. All LLM-generated content has been manually reviewed and tested. -- 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]
