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

triple async call support trace log #1282

Closed zhenjunMa closed 1 year ago

zhenjunMa commented 1 year ago

Motivation:

调用端使用triple协议发起异步调用(callback, future)时,不会打印trace日志,需要修复

Modification:

trace日志打印是基于trace事件的,因此需要在异步调用的过程中发送trace事件给trace模块

Result:

调用端使用triple协议发起异步调用(callback, future)时,可以打印trace日志

codecov[bot] commented 1 year ago

Codecov Report

Merging #1282 (8ca4d4a) into master (1e386ed) will decrease coverage by 0.04%. The diff coverage is 52.94%.

@@             Coverage Diff              @@
##             master    #1282      +/-   ##
============================================
- Coverage     72.01%   71.97%   -0.05%     
  Complexity      783      783              
============================================
  Files           414      414              
  Lines         17565    17616      +51     
  Branches       2740     2749       +9     
============================================
+ Hits          12650    12679      +29     
- Misses         3521     3535      +14     
- Partials       1394     1402       +8     
Impacted Files Coverage Δ
...sofa/rpc/transport/triple/TripleClientInvoker.java 61.95% <52.50%> (+0.30%) :arrow_up:
...fa/rpc/transport/triple/TripleClientTransport.java 70.83% <60.00%> (-0.84%) :arrow_down:
...java/com/alipay/sofa/rpc/module/LookoutModule.java 52.38% <0.00%> (-33.34%) :arrow_down:
...n/java/com/alipay/sofa/rpc/common/SofaConfigs.java 84.90% <0.00%> (-1.89%) :arrow_down:
...va/com/alipay/sofa/rpc/client/AbstractCluster.java 71.54% <0.00%> (+0.26%) :arrow_up:
...lipay/sofa/rpc/message/AbstractResponseFuture.java 57.01% <0.00%> (+0.87%) :arrow_up:
...ofa/rpc/message/bolt/BoltFutureInvokeCallback.java 79.24% <0.00%> (+7.54%) :arrow_up:

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