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

Do not truncate reported strings #174

Merged
merged 1 commit into from
May 9, 2017

Conversation

pavolloffay
Copy link
Member

resolves #173

@codecov-io
Copy link

Codecov Report

Merging #174 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #174      +/-   ##
============================================
- Coverage     79.58%   79.55%   -0.03%     
+ Complexity      448      445       -3     
============================================
  Files            76       76              
  Lines          1729     1722       -7     
  Branches        192      191       -1     
============================================
- Hits           1376     1370       -6     
+ Misses          267      266       -1     
  Partials         86       86
Impacted Files Coverage Δ Complexity Δ
...-core/src/main/java/com/uber/jaeger/Constants.java 0% <ø> (ø) 0 <0> (ø) ⬇️
...ber/jaeger/senders/zipkin/ThriftSpanConverter.java 83.33% <100%> (+1.85%) 19 <0> (ø) ⬇️
...reporters/protocols/JaegerThriftSpanConverter.java 91.8% <100%> (-0.51%) 17 <0> (-2)
.../uber/jaeger/samplers/RemoteControlledSampler.java 91.54% <0%> (-1.41%) 18% <0%> (-1%)

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 1c73d62...32d6a33. Read the comment docs.

@black-adder black-adder merged commit 284ec7f into jaegertracing:master May 9, 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.

zipkin.ThriftSpanConverter does not truncate tags
3 participants