Skip to content

Conversation

@BubbleCal
Copy link
Contributor

@BubbleCal BubbleCal commented Jan 7, 2026

  • Reuse hashmap
  • avoid frequent allocating by SmallVec

20% faster indexing

@codecov
Copy link

codecov bot commented Jan 7, 2026

Codecov Report

❌ Patch coverage is 87.00000% with 13 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
rust/lance-index/src/scalar/inverted/builder.rs 87.75% 4 Missing and 8 partials ⚠️
rust/lance-index/src/scalar/inverted/index.rs 50.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@BubbleCal BubbleCal force-pushed the yang/inverted-index-perf branch from 30cdd83 to ae99072 Compare January 7, 2026 14:26
@BubbleCal BubbleCal requested review from Xuanwo and westonpace January 8, 2026 04:56
@BubbleCal BubbleCal merged commit 4f7daa2 into main Jan 8, 2026
30 of 31 checks passed
@BubbleCal BubbleCal deleted the yang/inverted-index-perf branch January 8, 2026 08:36
@artemru
Copy link

artemru commented Jan 8, 2026

nice progress, cannot wait to test it!

jackye1995 pushed a commit to jackye1995/lance that referenced this pull request Jan 21, 2026
…rmat#5650)

- Reuse hashmap
- avoid frequent allocating by SmallVec 

20% faster indexing
yanghua pushed a commit that referenced this pull request Jan 21, 2026
Note:
1. Reposition merge_index_metadata so that Java dataset could directly
call mergeIndexMetadata for vector indexes.
2. Add `centroids` to IvfBuilderParams and `codebook` to PqBuilderParams
to make distrubitively creating vector indexes work
3. Add a new index package under tests to have index tests.
4. This PR update Cargo.lock due to the intruduction of #5650.
majin1102 added a commit to majin1102/lance that referenced this pull request Jan 23, 2026
…t#5664)

Note:
1. Reposition merge_index_metadata so that Java dataset could directly
call mergeIndexMetadata for vector indexes.
2. Add `centroids` to IvfBuilderParams and `codebook` to PqBuilderParams
to make distrubitively creating vector indexes work
3. Add a new index package under tests to have index tests.
4. This PR update Cargo.lock due to the intruduction of lance-format#5650.
vivek-bharathan pushed a commit to vivek-bharathan/lance that referenced this pull request Feb 2, 2026
…t#5664)

Note:
1. Reposition merge_index_metadata so that Java dataset could directly
call mergeIndexMetadata for vector indexes.
2. Add `centroids` to IvfBuilderParams and `codebook` to PqBuilderParams
to make distrubitively creating vector indexes work
3. Add a new index package under tests to have index tests.
4. This PR update Cargo.lock due to the intruduction of lance-format#5650.
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.

4 participants