Skip to content

[Timeseries-Storage] enable delta and deltaDelta encoding on raw column#17826

Open
deemoliu wants to merge 3 commits intoapache:masterfrom
deemoliu:qiaochu/fix-dd
Open

[Timeseries-Storage] enable delta and deltaDelta encoding on raw column#17826
deemoliu wants to merge 3 commits intoapache:masterfrom
deemoliu:qiaochu/fix-dd

Conversation

@deemoliu
Copy link
Contributor

@deemoliu deemoliu commented Mar 5, 2026

bugfix: Enable DELTA/DELTADELTA compression codecs on raw forward-index columns.

This change fixes the invalid codec error in tableConfig and marks these codecs as raw-index applicable.

Copy link
Contributor

@wirybeaver wirybeaver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov-commenter
Copy link

codecov-commenter commented Mar 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.58%. Comparing base (337faad) to head (040f667).
⚠️ Report is 30 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (337faad) and HEAD (040f667). Click for more details.

HEAD has 8 uploads less than BASE
Flag BASE (337faad) HEAD (040f667)
java-21 5 4
unittests 4 2
temurin 10 8
java-11 5 4
unittests2 2 0
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #17826      +/-   ##
============================================
- Coverage     63.23%   55.58%   -7.65%     
+ Complexity     1454      727     -727     
============================================
  Files          3183     2496     -687     
  Lines        191430   141131   -50299     
  Branches      29274    22505    -6769     
============================================
- Hits         121047    78452   -42595     
+ Misses        60943    56032    -4911     
+ Partials       9440     6647    -2793     
Flag Coverage Δ
custom-integration1 100.00% <ø> (ø)
integration 100.00% <ø> (ø)
integration1 100.00% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 55.54% <100.00%> (-7.67%) ⬇️
java-21 55.56% <100.00%> (-7.64%) ⬇️
temurin 55.58% <100.00%> (-7.65%) ⬇️
unittests 55.58% <100.00%> (-7.65%) ⬇️
unittests1 55.58% <100.00%> (-0.04%) ⬇️
unittests2 ?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants