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

#514 #656

Closed
Closed

Conversation

pedrohcavalcante
Copy link

Which problem is this PR solving?

Short description of the changes

  • renaming of JAEGER_SAMPLER_MANAGER_HOST_PORT to JAEGER_CONFIG_MGR_HOST_PORT

@codecov
Copy link

codecov bot commented Oct 9, 2019

Codecov Report

Merging #656 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff            @@
##             master    #656   +/-   ##
========================================
  Coverage      89.4%   89.4%           
  Complexity      563     563           
========================================
  Files            69      69           
  Lines          2076    2076           
  Branches        263     263           
========================================
  Hits           1856    1856           
  Misses          138     138           
  Partials         82      82
Impacted Files Coverage Δ Complexity Δ
.../src/main/java/io/jaegertracing/Configuration.java 93.18% <100%> (ø) 44 <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 02150c7...afdb767. Read the comment docs.

Pedro Cavalcante added 2 commits October 9, 2019 16:37
Signed-off-by: Pedro Cavalcante <pedroc@anchorloans.com>
Signed-off-by: Pedro Cavalcante <pedroc@anchorloans.com>
@yurishkuro
Copy link
Member

@yurishkuro
Copy link
Member

library is deprecated

@yurishkuro yurishkuro closed this Jan 10, 2022
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.

Rename JAEGER_SAMPLER_MANAGER_HOST_PORT to JAEGER_CONFIG_MGR_HOST_PORT
2 participants