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

Add support for references #241

Merged
merged 4 commits into from
Apr 27, 2019

Conversation

bhavin192
Copy link
Contributor

  • Checks if all the given references are valid
  • If the parent is not set then sets the first reference as parent of
    the span
  • Allows to pass one Reference object or list of them

Fixes #190

@codecov
Copy link

codecov bot commented Feb 25, 2019

Codecov Report

Merging #241 into master will increase coverage by 0.05%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #241      +/-   ##
==========================================
+ Coverage   94.58%   94.63%   +0.05%     
==========================================
  Files          25       25              
  Lines        1883     1902      +19     
  Branches      250      256       +6     
==========================================
+ Hits         1781     1800      +19     
  Misses         67       67              
  Partials       35       35
Impacted Files Coverage Δ
jaeger_client/tracer.py 99.31% <100%> (+0.02%) ⬆️
jaeger_client/thrift.py 95.87% <100%> (+0.58%) ⬆️
jaeger_client/span.py 96.33% <100%> (+0.03%) ⬆️

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 de9ad06...bdf4700. Read the comment docs.

- Checks if all the given references are valid
- If the parent is not set then sets the first reference as parent of
  the span
- Allows to pass one Reference object or list of them

Signed-off-by: Bhavin Gandhi <bhavin7392@gmail.com>
Signed-off-by: Bhavin Gandhi <bhavin7392@gmail.com>
Yuri Shkuro and others added 2 commits April 27, 2019 16:28
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.

Thanks! I made a small check to avoid memory allocation when there are no references.

@yurishkuro yurishkuro merged commit 9771cc9 into jaegertracing:master Apr 27, 2019
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.

Support Span References
2 participants