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

Remove reference to JAEGER_DISABLE_GLOBAL_TRACER #379

Merged

Conversation

Sharpie
Copy link
Contributor

@Sharpie Sharpie commented Apr 8, 2018

This commit removes JAEGER_DISABLE_GLOBAL_TRACER from the jaeger-core
docs. The functionality associated with this setting was removed in
#302.

Signed-off-by: Charlie Sharpsteen source@sharpsteen.net

This commit removes JAEGER_DISABLE_GLOBAL_TRACER from the jaeger-core
docs. The functionality associated with this setting was removed in
jaegertracing#302.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>
@codecov
Copy link

codecov bot commented Apr 8, 2018

Codecov Report

Merging #379 into master will decrease coverage by 0.16%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #379      +/-   ##
============================================
- Coverage     84.89%   84.73%   -0.17%     
  Complexity      607      607              
============================================
  Files            94       94              
  Lines          2410     2410              
  Branches        271      271              
============================================
- Hits           2046     2042       -4     
- Misses          272      276       +4     
  Partials         92       92
Impacted Files Coverage Δ Complexity Δ
...ain/java/com/uber/jaeger/senders/ThriftSender.java 75% <0%> (-4.55%) 10% <0%> (ø)
...java/com/uber/jaeger/reporters/RemoteReporter.java 84.7% <0%> (-2.36%) 8% <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 6d822c3...c6a597f. Read the comment docs.

Copy link
Member

@yurishkuro yurishkuro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution.

@yurishkuro yurishkuro merged commit e9c42ba into jaegertracing:master Apr 8, 2018
@Sharpie Sharpie deleted the remove-stray-global-tracer-docs branch April 9, 2018 02:02
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.

None yet

2 participants