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.82k stars 1.17k forks source link

add timeout to sofaRequest #1224

Closed EvenLjj closed 2 years ago

EvenLjj commented 2 years ago

Before timeout only set in com.alipay.remoting.rpc.protocol.RpcRequestCommand on bolt, now add timeout to com.alipay.sofa.rpc.core.request.SofaRequest when resolveTimeout.

codecov[bot] commented 2 years ago

Codecov Report

Merging #1224 (144199b) into master (3480c29) will increase coverage by 0.43%. The diff coverage is 91.94%.

@@             Coverage Diff              @@
##             master    #1224      +/-   ##
============================================
+ Coverage     71.60%   72.03%   +0.43%     
+ Complexity      830      779      -51     
============================================
  Files           408      410       +2     
  Lines         17241    17374     +133     
  Branches       2686     2702      +16     
============================================
+ Hits          12345    12515     +170     
+ Misses         3529     3477      -52     
- Partials       1367     1382      +15     
Impacted Files Coverage Δ
.../java/com/alipay/sofa/rpc/common/RpcConstants.java 100.00% <ø> (ø)
.../main/java/com/alipay/sofa/rpc/common/Version.java 100.00% <ø> (ø)
.../sofa/rpc/registry/local/DomainRegistryHelper.java 62.50% <62.50%> (ø)
...y/sofa/rpc/bootstrap/DefaultConsumerBootstrap.java 36.17% <71.42%> (+13.21%) :arrow_up:
...alipay/sofa/rpc/registry/local/DomainRegistry.java 94.16% <94.16%> (ø)
...va/com/alipay/sofa/rpc/client/AbstractCluster.java 70.75% <100.00%> (+0.07%) :arrow_up:
.../java/com/alipay/sofa/rpc/client/ProviderInfo.java 76.85% <100.00%> (+2.78%) :arrow_up:
...n/java/com/alipay/sofa/rpc/log/TimeWaitLogger.java 80.00% <0.00%> (-4.00%) :arrow_down:
...lipay/sofa/rpc/message/AbstractResponseFuture.java 56.14% <0.00%> (-0.88%) :arrow_down:
... and 6 more

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 8af82b2...144199b. Read the comment docs.