Skip to content

Commit de8473b

Browse files
committed
fix: update comment for arrow-ipc dependency usage in CompressionType and codec features
1 parent bc3535f commit de8473b

1 file changed

Lines changed: 3 additions & 7 deletions

File tree

datafusion/physical-plan/Cargo.toml

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -49,13 +49,9 @@ name = "datafusion_physical_plan"
4949
[dependencies]
5050
arrow = { workspace = true }
5151
arrow-data = { workspace = true }
52-
# Feature-only dep used only via the `arrow::ipc` re-export (there is no direct
53-
# `use arrow_ipc` in this crate): lz4 and zstd codec support required by
54-
# IPCStreamWriter in spill/mod.rs. These features must stay in sync with the
55-
# SpillCompression variants in datafusion-common. Declaring the dependency here
56-
# makes the contract explicit and local to the crate that owns spill, preventing
57-
# silent regressions if workspace-level arrow-ipc features are narrowed (see
58-
# #21917 for the documented regression guard around this coupling).
52+
# Spill IPC writes require lz4 and zstd codec support. Keep these features in
53+
# sync with the SpillCompression variants in datafusion-common so codec
54+
# availability is explicit in the crate that owns spill handling.
5955
arrow-ipc = { workspace = true, features = ["lz4", "zstd"] }
6056
arrow-ord = { workspace = true }
6157
arrow-schema = { workspace = true }

0 commit comments

Comments
 (0)