Skip to content

Conversation

@JervyShi
Copy link
Member

No description provided.

@codecov-io
Copy link

Codecov Report

Merging #210 into 5.4 will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                5.4     #210      +/-   ##
============================================
+ Coverage     71.27%   71.27%   +<.01%     
+ Complexity      957      956       -1     
============================================
  Files           341      341              
  Lines         14213    14213              
  Branches       2286     2286              
============================================
+ Hits          10130    10131       +1     
- Misses         2887     2888       +1     
+ Partials       1196     1194       -2
Impacted Files Coverage Δ Complexity Δ
.../main/java/com/alipay/sofa/rpc/common/Version.java 100% <ø> (ø) 0 <0> (ø) ⬇️
...m/alipay/sofa/rpc/module/FaultToleranceModule.java 66.66% <0%> (-26.67%) 4% <0%> (-1%)
.../sofa/rpc/bootstrap/DefaultClientProxyInvoker.java 87.09% <0%> (-1.62%) 0% <0%> (ø)
.../sofa/rpc/registry/zk/ZookeeperRegistryHelper.java 75% <0%> (-1.43%) 0% <0%> (ø)
...ay/sofa/rpc/client/AllConnectConnectionHolder.java 62.84% <0%> (-0.28%) 0% <0%> (ø)
...com/alipay/sofa/rpc/context/RpcRuntimeContext.java 90.24% <0%> (+1.21%) 0% <0%> (ø) ⬇️
...lipay/sofa/rpc/message/AbstractResponseFuture.java 55.35% <0%> (+1.78%) 0% <0%> (ø) ⬇️
.../main/java/com/alipay/sofa/rpc/tracer/Tracers.java 42.3% <0%> (+2.56%) 0% <0%> (ø) ⬇️
...pay/sofa/rpc/transport/ClientTransportFactory.java 78.46% <0%> (+6.15%) 0% <0%> (ø) ⬇️

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 95f752b...9fa533a. Read the comment docs.

@leizhiyuan leizhiyuan self-requested a review June 29, 2018 13:29
@leizhiyuan leizhiyuan merged commit c7017d1 into sofastack:5.4 Jun 29, 2018
@JervyShi JervyShi deleted the update_5.4.2 branch June 29, 2018 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants