-
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: support gRPC cancellation #4092
feat: support gRPC cancellation #4092
Conversation
db27e7c
to
61f85a8
Compare
76f4ef3
to
2d2d75d
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4092 +/- ##
==========================================
- Coverage 85.48% 85.15% -0.33%
==========================================
Files 993 994 +1
Lines 173724 173825 +101
==========================================
- Hits 148511 148028 -483
- Misses 25213 25797 +584 |
2d2d75d
to
cf968e2
Compare
e252212
to
1099364
Compare
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.
My 5th time forgetting about the relationship of Datanode
, RegionInvoker
and RegionServer
Me too, cc @WenyXu |
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
This PR enables gRPC server cancellation support in
RegionServer
andGreptimeService
.Closes #3964
Checklist