Closed zyy17 closed 4 days ago
The changes in the src/mito2/src/region/options.rs
module add a new serialization function serialize_ignore_column_ids
for handling column IDs in the InvertedIndexOptions
struct. Additionally, a corresponding test function test_region_options_serde
has been introduced to verify the serialization and deserialization processes of RegionOptions
.
File Path | Change Summary |
---|---|
.../mito2/src/region/options.rs | Added serialize_ignore_column_ids function and corresponding #[serde(serialize_with = "...")] attribute. Introduced test_region_options_serde function to test serialization and deserialization. |
sequenceDiagram
actor Developer
participant CodeBase
Developer->>CodeBase: Add serialize_ignore_column_ids to InvertedIndexOptions
CodeBase->>Developer: Update applied
Developer->>CodeBase: Add test_region_options_serde
CodeBase->>Developer: Test function created
Developer->>CodeBase: Serialize RegionOptions
CodeBase->>Developer: Serialized data
Developer->>CodeBase: Deserialize RegionOptions
CodeBase->>Developer: Deserialized data
Amidst the rustling code so bright,
A function springs to life, a guiding light.
In columns' whispers hear the cheer,
Options serene, both far and near. 🌟
Tested truths, in bytes unfold,
Region’s tale in code retold. 📜
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?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.59%. Comparing base (
ef935a1
) to head (9f97f82
).
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?
Add
serialize_ignore_column_ids()
to fix deserialize region options failed from json stringChecklist
Summary by CodeRabbit