-
Notifications
You must be signed in to change notification settings - Fork 328
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
refactor: set default metasrv procedure retry times to 12 #2242
refactor: set default metasrv procedure retry times to 12 #2242
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #2242 +/- ##
===========================================
- Coverage 85.00% 84.64% -0.37%
===========================================
Files 704 706 +2
Lines 115202 115557 +355
===========================================
- Hits 97929 97809 -120
- Misses 17273 17748 +475 |
94961e8
to
5f56aaa
Compare
5f56aaa
to
f12e02c
Compare
Suggestion: add comments about the consequence of the retry strategy. For example, max time a procedure can last; retry interval etc. |
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
f12e02c
to
a09b70c
Compare
I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Set default metasrv procedure retry times to 12
Checklist
Refer to a related PR or issue link (optional)