Skip to content

Conversation

@iosmanthus
Copy link
Member

Signed-off-by: iosmanthus myosmanthustree@gmail.com

What problem does this PR solve?

Issue Number: close #572

Signed-off-by: iosmanthus <myosmanthustree@gmail.com>
Copy link
Member

@zz-jason zz-jason left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zz-jason zz-jason enabled auto-merge (squash) March 24, 2022 07:55
@codecov
Copy link

codecov bot commented Mar 24, 2022

Codecov Report

Merging #573 (a12a6f6) into master (6cbf56a) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #573      +/-   ##
============================================
- Coverage     33.21%   33.13%   -0.09%     
+ Complexity     1336     1332       -4     
============================================
  Files           270      270              
  Lines         17131    17131              
  Branches       1950     1950              
============================================
- Hits           5690     5676      -14     
- Misses        10831    10848      +17     
+ Partials        610      607       -3     
Impacted Files Coverage Δ
...ty/handler/codec/http2/Http2ConnectionHandler.java 49.38% <0.00%> (-1.96%) ⬇️
...rc/main/java/io/grpc/netty/NettyClientHandler.java 55.81% <0.00%> (-1.30%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6cbf56a...a12a6f6. Read the comment docs.

@iosmanthus
Copy link
Member Author

/merge

@ti-srebot
Copy link
Collaborator

/run-all-tests

@ti-srebot
Copy link
Collaborator

/run-all-tests

@iosmanthus
Copy link
Member Author

/merge

@ti-srebot
Copy link
Collaborator

/run-all-tests

@zz-jason zz-jason merged commit ef16787 into tikv:master Mar 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Syntax issues in GitHub template

4 participants