Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change Footer version handling, Make compression dynamic #1060

Merged
merged 10 commits into from
May 28, 2021

Conversation

PSeitz
Copy link
Contributor

@PSeitz PSeitz commented May 24, 2021

Change Footer version handling
Simplify version handling by switching to JSON instead of binary serialization.
fixes #1058

Make compression dynamic
Instead of choosing the compression during compile time via a feature flag, you can now have multiple compression algorithms enabled and decide during runtime which one to choose via IndexSettings. Changing the compression algorithm on an index is also supported. The information which algorithm was used in the doc store is stored in the DocStoreFooter. The default is the lz4 block format.
fixes #904

Change Footer version handling
Simplify version handling by switching to JSON instead of binary serialization.
fixes quickwit-oss#1058

Make compression dynamic
Instead of choosing the compression during compile time via a feature flag, you can now have multiple compression algorithms enabled and decide during runtime which one to choose via IndexSettings. Changing the compression algorithm on an index is also supported. The information which algorithm was used in the doc store is stored in the DocStoreFooter. The default is the lz4 block format.
fixes quickwit-oss#904
Handle merging of different compressors
Fix feature flag names
Add doc store test for all compressors
@PSeitz PSeitz requested a review from fulmicoton May 24, 2021 11:55
src/core/index_meta.rs Outdated Show resolved Hide resolved
Copy link
Collaborator

@fulmicoton fulmicoton left a comment

Choose a reason for hiding this comment

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

I added some minor comments.

Mostly renaming.

@PSeitz
Copy link
Contributor Author

PSeitz commented May 25, 2021

Should be fine now, except removing the lz-frame stuff

Edit also removed the lz4 frame stuff

use docstore_compression default fallback in order to parse old versions of meta.json
rename lz4-block-compression to lz4-compression
@PSeitz PSeitz requested a review from fulmicoton May 26, 2021 05:31
src/directory/footer.rs Outdated Show resolved Hide resolved
src/store/mod.rs Outdated Show resolved Hide resolved
Copy link
Collaborator

@fulmicoton fulmicoton left a comment

Choose a reason for hiding this comment

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

Sorry I added a bunch of extra comments

@fulmicoton fulmicoton merged commit 8d32c3b into quickwit-oss:main May 28, 2021
This was referenced Feb 18, 2022
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.

Change Footer versioning handling Dynamic compression format
2 participants