-
Notifications
You must be signed in to change notification settings - Fork 329
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
feat(mito): Implement open and close for mito2 regions #2052
feat(mito): Implement open and close for mito2 regions #2052
Conversation
e94039c
to
9f6ce4b
Compare
Codecov Report
@@ Coverage Diff @@
## develop #2052 +/- ##
===========================================
- Coverage 85.17% 84.86% -0.31%
===========================================
Files 672 673 +1
Lines 105988 106327 +339
===========================================
- Hits 90277 90239 -38
- Misses 15711 16088 +377 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…2052) * feat: add close request * feat: handle close and open request * feat: Implement open * test: add TestEnv::new * feat: close region/engine and test * style: fix clippy * style: import log macros * docs: update docs * docs: add mermaid for manifest manager
I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
This PR implements
open_region
andclose_region
for mito2 engine.It also stops all regions while stopping the engine.
Checklist
Refer to a related PR or issue link (optional)