Closed zyy17 closed 3 weeks ago
In the CompactionScheduler
implementation in compaction.rs
, a new field region_dir
has been introduced to the CompactionRegion
struct. This field is populated with the value obtained from access_layer.region_dir().to_string()
. Additionally, the field assignments within the open_compaction_region
function have been reordered.
File | Change Summary |
---|---|
src/mito2/src/compaction.rs | Added region_dir: String field to CompactionRegion struct. Reordered field assignments in open_compaction_region function. |
sequenceDiagram
participant User
participant CompactionScheduler
participant AccessLayer
participant CompactionRegion
User ->> CompactionScheduler: Initiate compaction
CompactionScheduler ->> AccessLayer: Retrieve region_dir
AccessLayer -->> CompactionScheduler: region_dir value
CompactionScheduler ->> CompactionRegion: new(region_dir)
🐇 Amidst the code where changes gleam, A
region_dir
joins the team, In compaction dance, it finds its place, With reordered steps, it sets the pace. Oh, how it brings a fresh new scheme, For files to follow, and data's dream. 🌟
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?
Attention: Patch coverage is 14.28571%
with 6 lines
in your changes missing coverage. Please review.
Project coverage is 84.80%. Comparing base (
1e52ba3
) to head (1d57f2a
). Report is 5 commits behind head on main.
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
region_dir
in CompactionRegion.Checklist
Summary by CodeRabbit
region_dir
field to better manage region directories.