-
Notifications
You must be signed in to change notification settings - Fork 316
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: impl migrate_region and procedure_state SQL function #3325
feat: impl migrate_region and procedure_state SQL function #3325
Conversation
1192404
to
f864e53
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3325 +/- ##
==========================================
- Coverage 85.28% 85.25% -0.03%
==========================================
Files 884 886 +2
Lines 144256 144589 +333
==========================================
+ Hits 123028 123270 +242
- Misses 21228 21319 +91 |
f864e53
to
0518471
Compare
…der greptime catalog
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
Just to confirm, is the unit of the |
Yes, I think seconds unit is enough. |
I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
It's the following PR of #3236 and #3278 .
Improves:
Implements:
migrate_region(region_id, from_peer, to_peer, [timeout])
to migrate region betweeen datanodes.procedure_state(procedure_id)
to query procedure state.Note: for security reasons, these two functions only can be called under the
greptime
catalog.For example:
The SQL finds the datanode peer id and region id by table name.
It will return the procedure id such as
538b7476-9f79-4e50-aa9c-b1de90710839
.It will return the procedure state in JSON
Checklist
Refer to a related PR or issue link (optional)
#2931 #3161 #2921