-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[fix] Update gRPC to 1.75.0 #24813
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
[fix] Update gRPC to 1.75.0 #24813
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #24813 +/- ##
============================================
- Coverage 74.50% 74.24% -0.27%
+ Complexity 33895 33392 -503
============================================
Files 1912 1912
Lines 149072 149072
Branches 17299 17299
============================================
- Hits 111071 110674 -397
- Misses 29218 29553 +335
- Partials 8783 8845 +62
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
(cherry picked from commit 8b42752)
|
I removed the label for 3.0.x since upgrading gRPC for 3.0.x is most likely not compatible with BookKeeper's generated stubs for stream storage in BookKeeper 4.16.x branch. In BookKeeper 4.17.x branch, gRPC 1.72.0 is used and 4.17.2 was released with gRPC 1.72.0 which is most likely compatible with gRPC 1.75.0. |
(cherry picked from commit 8b42752)
Motivation
Modifications
Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)
Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes
Documentation
docdoc-requireddoc-not-neededdoc-completeMatching PR in forked repository
PR in forked repository: