guo0693 commented 5 years ago

upgrade opentracing to 0.32.0
upgrade jaeger to 1.1.1
fix the infinite loop bug when closing tracing context due to the NoopScopeManager always return non-null instance Base on the comment: https://code.uberinternal.com/D2999387?id=9423683#inline-21546995

CLAassistant commented 5 years ago

All committers have signed the CLA.

guo0693 commented 5 years ago

tchannel seems to be using google maven central, java monorepo is not publishing to that artifactory, going to figure this out.

codecov-io commented 5 years ago

Codecov Report

Merging #199 into master will increase coverage by 0.03%. The diff coverage is 91.67%.

@@             Coverage Diff              @@
##             master     #199      +/-   ##
============================================
+ Coverage     71.95%   71.99%   +0.03%     
  Complexity       10       10              
============================================
  Files            87       87              
  Lines          2660     2663       +3     
  Branches        320      319       -1     
============================================
+ Hits           1914     1917       +3     
  Misses          525      525              
  Partials        221      221

Impacted Files	Coverage Δ	Complexity Δ
.../com/uber/tchannel/tracing/OpenTracingContext.java	`90.48% <91.67%> (+1.59%)`	`0 <0> (ø)`	:arrow_down:

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 009cf1e...007157e. Read the comment docs.

uber / tchannel-java

Upgrade opentracing & jaeger lib #199

Codecov Report