feat: implement sync_region for mito engine#5765
feat: implement sync_region for mito engine#5765fengjiachun merged 10 commits intoGreptimeTeam:mainfrom
sync_region for mito engine#5765Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
395fb4d to
93dc8e2
Compare
* chore: upgrade proto to `2d52b` * feat: add `SyncRegion` to `WorkerRequest` * feat: impl `sync_region` for `Engine` trait * test: add tests * chore: fmt code * chore: upgrade proto * chore: unify `RegionLeaderState` and `RegionFollowerState` * chore: check immutable memtable * chore: fix clippy * chore: apply suggestions from CR
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
#5739
What's changed and what's your intention?
This PR implements the sync region for the mito engine, It adds a
sync_regionmethod to MitoEngine, which allows a follower region's manifest to update to at least a specified manifest version received from the metasrv.PR Checklist
Please convert it to a draft if some of the following conditions are not met.