Closed v0y4g3r closed 2 days ago
[!NOTE]
Reviews paused
Use the following commands to manage reviews:
@coderabbitai resume
to resume automatic reviews.@coderabbitai review
to trigger a single review.
The recent changes modify how the InvertedIndexReader
and associated methods handle offsets and sizes. Instead of passing a metadata structure, methods now directly take offset and size parameters. This simplifies the interfaces of fst
and bitmap
methods and adjusts related logic and tests accordingly.
File Path | Change Summary |
---|---|
src/index/src/inverted_index/format/reader.rs |
Updated InvertedIndexReader trait methods to accept direct offset and size parameters instead of metadata structure. |
src/index/src/inverted_index/format/reader/blob.rs |
Adjusted fst and bitmap method implementations to use direct offset and size. Updated test calls to match the new signature. |
src/index/src/inverted_index/format/writer/blob.rs |
Modified calls to fst and bitmap methods in the writer to include base offsets and sizes. |
src/index/src/inverted_index/search/fst_values_mapper.rs |
Updated method call in FstValuesMapper to use the new offset and size parameters. Simplified test setup closure parameters. |
src/index/src/inverted_index/search/index_apply/...rs |
Changed method calls in PredicatesIndexApplier to utilize new offsets and sizes, updated test signatures for fetching fst . |
In the fields where data blooms, We shift offsets and sizes for new rooms. No longer bound by meta’s hold, Directly we seek, our methods bold. With simplified code and tests anew, Our indexing reads, swift and true. 🐰📘
[!TIP]
Early access features: enabled
We are currently testing the following features in early access: - **OpenAI `gpt-4o` model for code reviews and chat**: OpenAI claims that this model is better at understanding and generating code than the previous models. We seek your feedback over the next few weeks before making it generally available. Note: - You can enable or disable early access features from the CodeRabbit UI or by updating the CodeRabbit configuration file. - Please join our [Discord Community](https://discord.com/invite/GsXnASn26c) to provide feedback and report issues. - OSS projects are currently opted into early access features by default.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@coderabbitai pause
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.69%. Comparing base (
f2c08b8
) to head (144aad0
).
@zhongzc PTAL
…acilliate caching
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
Change methods signatures of
InvertedIndexWriter
to offset/size so that we can add a cache layer from arguments provided.Checklist
Summary by CodeRabbit
Refactor
fst
andbitmap
methods to improve performance by using direct offsets and sizes instead of metadata references.Tests