sofastack / sofa-rpc

SOFARPC is a high-performance, high-extensibility, production-level Java RPC framework.
https://www.sofastack.tech/sofa-rpc/docs/Home
Apache License 2.0
3.81k stars 1.17k forks source link

Fix log connection get error #1243

Closed MingJunDuan closed 1 year ago

MingJunDuan commented 1 year ago

Motivation:

Fix log connection get exception #1240

Modification:

Describe the idea and modifications you've done.

Result:

Fixes #.

If there is no issue then describe the changes introduced by this PR.

codecov[bot] commented 1 year ago

Codecov Report

Merging #1243 (ce643fd) into master (169016c) will increase coverage by 0.03%. The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1243      +/-   ##
============================================
+ Coverage     72.04%   72.08%   +0.03%     
  Complexity      779      779              
============================================
  Files           410      410              
  Lines         17376    17378       +2     
  Branches       2703     2703              
============================================
+ Hits          12519    12527       +8     
+ Misses         3475     3473       -2     
+ Partials       1382     1378       -4     
Impacted Files Coverage Δ
.../java/com/alipay/sofa/rpc/common/RpcConstants.java 100.00% <ø> (ø)
.../main/java/com/alipay/sofa/rpc/common/Version.java 100.00% <ø> (ø)
...ipay/sofa/rpc/codec/bolt/SofaRpcSerialization.java 76.19% <100.00%> (+0.17%) :arrow_up:
...ansport/bolt/ReuseBoltClientConnectionManager.java 69.56% <100.00%> (-2.90%) :arrow_down:
...n/java/com/alipay/sofa/rpc/common/SofaConfigs.java 84.90% <0.00%> (-1.89%) :arrow_down:
...om/alipay/sofa/rpc/server/triple/TripleServer.java 74.86% <0.00%> (-1.07%) :arrow_down:
...ay/sofa/rpc/client/AllConnectConnectionHolder.java 60.76% <0.00%> (+0.25%) :arrow_up:
...ipay/sofa/rpc/tracer/sofatracer/RpcSofaTracer.java 91.06% <0.00%> (+0.74%) :arrow_up:
...ipay/sofa/rpc/server/bolt/BoltServerProcessor.java 69.59% <0.00%> (+4.72%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.