Skip to content

VarBinViewBuilder: support appending buffers directly into the builder storage - #9235

Merged
robert3005 merged 3 commits into
developfrom
rk/varbinview-append-buffer
Aug 7, 2026
Merged

VarBinViewBuilder: support appending buffers directly into the builder storage#9235
robert3005 merged 3 commits into
developfrom
rk/varbinview-append-buffer

Skip null bytes in VarBinView buffer compaction

a9a9bdd
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 7, 2026 in 1s

82.59% (-4.74%) compared to 0f0390a

View this Pull Request on Codecov

82.59% (-4.74%) compared to 0f0390a

Details

Codecov Report

❌ Patch coverage is 92.10526% with 33 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.59%. Comparing base (0f0390a) to head (a9a9bdd).
⚠️ Report is 2 commits behind head on develop.

Files with missing lines Patch % Lines
encodings/zstd/src/array.rs 40.90% 13 Missing ⚠️
vortex-array/src/builders/varbinview.rs 95.02% 11 Missing ⚠️
vortex-array/src/arrays/varbin/vtable/mod.rs 33.33% 4 Missing ⚠️
vortex-array/src/arrays/varbin/vtable/canonical.rs 25.00% 3 Missing ⚠️
vortex-array/src/arrays/varbinview/build_views.rs 99.25% 1 Missing ⚠️
vortex-cuda/src/kernel/encodings/fsst.rs 0.00% 1 Missing ⚠️

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
View more details on Codecov