lance
lance copied to clipboard
fix: various improvements for 2.1 take, put FSL back into the compressors
Previously we moved all FSL handling out into the rep/def levels. Now we're back at it again, but this time we've put the FSL handling back into the compressors. It turns out we really don't want to store rep/def levels for every element in a vector.
There are also some minor perf. improvements to bitpacking and we disable bitpacking if the input is really small (mainly for testing purposes).
Another significant change is that the miniblock encoding now allows for a variable number of buffers instead of requiring things to be zipped into one.
Codecov Report
Attention: Patch coverage is 80.80960% with 256 lines in your changes missing coverage. Please review.
Project coverage is 78.82%. Comparing base (
a6101e5) to head (f9e179b).
Additional details and impacted files
@@ Coverage Diff @@
## main #3450 +/- ##
========================================
Coverage 78.81% 78.82%
========================================
Files 251 251
Lines 92821 93752 +931
Branches 92821 93752 +931
========================================
+ Hits 73159 73900 +741
- Misses 16686 16869 +183
- Partials 2976 2983 +7
| Flag | Coverage Δ | |
|---|---|---|
| unittests | 78.82% <80.80%> (+<0.01%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Superceded by https://github.com/lancedb/lance/pull/3488