dubbo icon indicating copy to clipboard operation
dubbo copied to clipboard

[3.0] run timeout task in separate thread to avoid the worker thread blocking

Open zrlw opened this issue 4 years ago • 1 comments

What is the purpose of the change

#8949

zrlw avatar Sep 29 '21 09:09 zrlw

Codecov Report

Merging #8954 (dce59a6) into 3.0 (858f8a9) will increase coverage by 0.01%. The diff coverage is 84.21%.

Impacted file tree graph

@@             Coverage Diff              @@
##                3.0    #8954      +/-   ##
============================================
+ Coverage     64.70%   64.71%   +0.01%     
+ Complexity      329      328       -1     
============================================
  Files          1206     1206              
  Lines         51844    51859      +15     
  Branches       7716     7718       +2     
============================================
+ Hits          33546    33563      +17     
+ Misses        14685    14683       -2     
  Partials       3613     3613              
Impacted Files Coverage Δ
...rg/apache/dubbo/common/timer/HashedWheelTimer.java 85.29% <84.21%> (+0.41%) :arrow_up:
.../remoting/transport/netty4/NettyServerHandler.java 58.13% <0.00%> (-9.31%) :arrow_down:
.../common/threadpool/serial/SerializingExecutor.java 70.37% <0.00%> (-7.41%) :arrow_down:
...e/dubbo/remoting/transport/netty/NettyChannel.java 55.68% <0.00%> (-4.55%) :arrow_down:
...pache/dubbo/remoting/transport/AbstractClient.java 58.57% <0.00%> (-1.43%) :arrow_down:
...vent/listener/ServiceInstancesChangedListener.java 79.91% <0.00%> (+0.41%) :arrow_up:
...ng/zookeeper/curator5/Curator5ZookeeperClient.java 66.66% <0.00%> (+0.96%) :arrow_up:
...exchange/support/header/HeaderExchangeHandler.java 68.14% <0.00%> (+1.76%) :arrow_up:
.../registry/zookeeper/ZookeeperServiceDiscovery.java 85.88% <0.00%> (+2.35%) :arrow_up:
... and 3 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 858f8a9...dce59a6. Read the comment docs.

codecov-commenter avatar Sep 29 '21 10:09 codecov-commenter