[ISSUE #7199] grpcClientChannel header add null judgement#7238
[ISSUE #7199] grpcClientChannel header add null judgement#7238Oliverwqcwrw merged 6 commits intoapache:developapache/rocketmq:developfrom weihubeats:weihubeats/bugfix_grpcClient_nullweihubeats/rocketmq:weihubeats/bugfix_grpcClient_nullCopy head branch name to clipboard
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #7238 +/- ##
=============================================
- Coverage 42.97% 42.94% -0.03%
+ Complexity 9670 9661 -9
=============================================
Files 1161 1161
Lines 83926 83926
Branches 10898 10898
=============================================
- Hits 36069 36045 -24
- Misses 43380 43403 +23
- Partials 4477 4478 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@Oliverwqcwrw Can you look at this pr? |
Oliverwqcwrw
left a comment
There was a problem hiding this comment.
Although I don't think the header should be null in theory, it is necessary for the robustness of the code to add this null judgment.
|
@RongtongJin Can you help merge this pr |
|
@Oliverwqcwrw Can you help merge this pr |
…bugfix_grpcClient_null
|
@RongtongJin Can you help merge this pr |
|
This PR is stale because it has been open for 365 days with no activity. It will be closed in 3 days if no further activity occurs. If you wish not to mark it as stale, please leave a comment in this PR. |
|
@Oliverwqcwrw Can you look at this pr? |
Thanks for your contribution |
…he#7238) adding a null judgement
#7199