Skip to content
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

[type:optimize] upgrade grpc #4841

Merged
merged 37 commits into from
Jul 26, 2023
Merged

[type:optimize] upgrade grpc #4841

merged 37 commits into from
Jul 26, 2023

Conversation

misaya295
Copy link
Contributor

@misaya295 misaya295 commented Jul 13, 2023

Fixes #4826

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@codecov-commenter
Copy link

codecov-commenter commented Jul 13, 2023

Codecov Report

Merging #4841 (8d8b8af) into master (2a416d5) will increase coverage by 0.03%.
Report is 1 commits behind head on master.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #4841      +/-   ##
============================================
+ Coverage     64.01%   64.04%   +0.03%     
- Complexity     8311     8314       +3     
============================================
  Files          1180     1180              
  Lines         34864    34865       +1     
  Branches       3115     3115              
============================================
+ Hits          22318    22330      +12     
+ Misses        10775    10762      -13     
- Partials       1771     1773       +2     
Files Changed Coverage Δ
...e/shenyu/plugin/grpc/client/GrpcClientBuilder.java 68.00% <100.00%> (+1.33%) ⬆️

... and 6 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@moremind
Copy link
Member

pending

@loongs-zhang loongs-zhang added the type: dependencies Pull requests that update a dependency file label Jul 14, 2023
@moremind moremind added this to the 2.6.0 milestone Jul 24, 2023
@yu199195
Copy link
Member

@misaya295 can you update license file?

@misaya295
Copy link
Contributor Author

@misaya295 can you update license file?

done

@yu199195 yu199195 self-requested a review July 26, 2023 02:58
moremind
moremind previously approved these changes Jul 26, 2023
yu199195
yu199195 previously approved these changes Jul 26, 2023
@yu199195 yu199195 closed this Jul 26, 2023
@yu199195 yu199195 reopened this Jul 26, 2023
@yu199195 yu199195 dismissed stale reviews from moremind and themself via 8d8b8af July 26, 2023 03:03
@yu199195 yu199195 merged commit e3cf9ed into apache:master Jul 26, 2023
75 checks passed
@misaya295 misaya295 deleted the grpc branch August 17, 2023 04:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin: grpc type: dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use ectd as a register to report an error
6 participants