Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

Upgrade to opentracing-java 0.21.0 #137

Merged
merged 1 commit into from
Apr 13, 2017

Conversation

objectiser
Copy link
Contributor

@objectiser objectiser commented Apr 13, 2017

and handle PEER_PORT change from short to int

@CLAassistant
Copy link

CLAassistant commented Apr 13, 2017

CLA assistant check
All committers have signed the CLA.

@codecov-io
Copy link

Codecov Report

Merging #137 into master will increase coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #137      +/-   ##
============================================
+ Coverage     35.96%   35.99%   +0.03%     
- Complexity      587      589       +2     
============================================
  Files            87       93       +6     
  Lines          6298     6301       +3     
  Branches       1046     1046              
============================================
+ Hits           2265     2268       +3     
- Misses         3852     3854       +2     
+ Partials        181      179       -2
Impacted Files Coverage Δ Complexity Δ
...r/jaeger/httpclient/TracingRequestInterceptor.java 84.61% <100%> (ø) 5 <0> (ø) ⬇️
...ain/java/com/uber/jaeger/context/TracingUtils.java 50% <0%> (-5.56%) 2% <0%> (-3%)
...m/uber/jaeger/httpclient/ClientRequestCarrier.java 83.33% <0%> (ø) 2% <0%> (-2%) ⬇️
...ain/java/com/uber/jaeger/metrics/StatsFactory.java
...er/jaeger/filters/jaxrs2/ClientRequestCarrier.java 83.33% <0%> (ø) 2% <0%> (?)
...a/com/uber/jaeger/filters/jaxrs2/TracingUtils.java 60% <0%> (ø) 3% <0%> (?)
.../java/com/uber/jaeger/dropwizard/StatsFactory.java 100% <0%> (ø) 4% <0%> (?)
...java/com/uber/jaeger/filters/jaxrs2/Constants.java 0% <0%> (ø) 0% <0%> (?)
...-core/src/main/java/com/uber/jaeger/Constants.java 0% <0%> (ø) 0% <0%> (?)
...java/com/uber/jaeger/dropwizard/Configuration.java 50% <0%> (ø) 3% <0%> (?)
... 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 54852c1...3ce0c19. Read the comment docs.

@objectiser
Copy link
Contributor Author

This update was required to enable Jaeger's java Tracer to be used with the Apache Camel camel-example-opentracing example.

@pavolloffay
Copy link
Member

Could you please do release of 0.18.0 once this is merged?

@yurishkuro yurishkuro changed the title Upgrade to opentracing-java 0.21.0 and handle PEER_PORT change from s… Upgrade to opentracing-java 0.21.0 Apr 13, 2017
@yurishkuro yurishkuro merged commit dce5c83 into jaegertracing:master Apr 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants