Skip to content

Support raw inverted index on multi-value columns#18295

Open
raghavyadav01 wants to merge 1 commit intoapache:masterfrom
raghavyadav01:raw-inverted-index-mv-column
Open

Support raw inverted index on multi-value columns#18295
raghavyadav01 wants to merge 1 commit intoapache:masterfrom
raghavyadav01:raw-inverted-index-mv-column

Conversation

@raghavyadav01
Copy link
Copy Markdown
Collaborator

Add Support for MV columns in raw Inverted index

Tests

  • Added testInvertedIndexHandlerRawMVColumn in RawValueBitmapInvertedIndexTest

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 22, 2026

Codecov Report

❌ Patch coverage is 89.65517% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.61%. Comparing base (78e38eb) to head (331af95).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...dex/loader/invertedindex/InvertedIndexHandler.java 89.65% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #18295      +/-   ##
============================================
+ Coverage     63.58%   63.61%   +0.02%     
  Complexity     1659     1659              
============================================
  Files          3245     3245              
  Lines        197441   197469      +28     
  Branches      30564    30570       +6     
============================================
+ Hits         125536   125611      +75     
+ Misses        61856    61813      -43     
+ Partials      10049    10045       -4     
Flag Coverage Δ
custom-integration1 100.00% <ø> (ø)
integration 100.00% <ø> (ø)
integration1 100.00% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 63.58% <89.65%> (+8.07%) ⬆️
java-21 63.58% <89.65%> (+0.02%) ⬆️
temurin 63.61% <89.65%> (+0.02%) ⬆️
unittests 63.60% <89.65%> (+0.02%) ⬆️
unittests1 55.54% <0.00%> (+<0.01%) ⬆️
unittests2 35.07% <89.65%> (+0.03%) ⬆️

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.

@xiangfu0
Copy link
Copy Markdown
Contributor

I feel we should get this in to support raw forward index+ inverted index with dictionary: #17269

cc: @Jackie-Jiang

@Jackie-Jiang
Copy link
Copy Markdown
Contributor

I feel we should get this in to support raw forward index+ inverted index with dictionary: #17269

+1. That is the cleaner solution.

@raghavyadav01 Is this feature needed urgently? We'll remove this eventually.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants