-
Notifications
You must be signed in to change notification settings - Fork 332
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: add CatchupRequest to engine #2939
feat: add CatchupRequest to engine #2939
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #2939 +/- ##
===========================================
- Coverage 85.82% 85.37% -0.45%
===========================================
Files 775 777 +2
Lines 124230 125219 +989
===========================================
+ Hits 106621 106910 +289
- Misses 17609 18309 +700 |
202a620
to
919f71d
Compare
1f2d1ab
to
bc962d6
Compare
set_writable_gracefully
for engineb2757ae
to
206982c
Compare
@MichaelScofield @evenyag PTAL |
0da0b6a
to
a961a00
Compare
feat: reopen region before replay if need
a961a00
to
2c57764
Compare
@MichaelScofield @evenyag PTAL |
07d5c0f
to
bb4c21a
Compare
@MichaelScofield @evenyag PTAL |
bb4c21a
to
981c306
Compare
Co-authored-by: Yingwen <[email protected]>
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
I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Add CatchupRequest; it makes a readonly region to catch up to leader region changes.
Checklist
Refer to a related PR or issue link (optional)
#2700