You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
PR #5038 reuses the Arrow IPC write context across shuffle blocks, but Comet currently pins Arrow 58.4.0. In that version, CompressionContext only owns an optional zstd compressor. Comet does not enable Arrow IPC buffer compression, and the FlatBufferBuilder is still allocated separately for every batch, so the merged lifetime change provides no allocation savings on current main.
Starting with Arrow 59.1, IpcWriteContext owns the reusable FlatBufferBuilder, making the context reuse introduced by #5038 effective.
The broader DataFusion/Arrow dependency upgrade is already tracked by #4865 and draft PR #5262. This issue tracks validating the deferred shuffle performance benefit after that upgrade lands.
Describe the potential solution
Close this issue when:
Comet uses Arrow 59.1 or newer through a compatible DataFusion upgrade.
What is the problem the feature request solves?
PR #5038 reuses the Arrow IPC write context across shuffle blocks, but Comet currently pins Arrow 58.4.0. In that version,
CompressionContextonly owns an optional zstd compressor. Comet does not enable Arrow IPC buffer compression, and theFlatBufferBuilderis still allocated separately for every batch, so the merged lifetime change provides no allocation savings on currentmain.Starting with Arrow 59.1,
IpcWriteContextowns the reusableFlatBufferBuilder, making the context reuse introduced by #5038 effective.The broader DataFusion/Arrow dependency upgrade is already tracked by #4865 and draft PR #5262. This issue tracks validating the deferred shuffle performance benefit after that upgrade lands.
Describe the potential solution
Close this issue when:
Comet's outer per-block compression encoders are independent and outside this issue's scope.
Additional context