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

Make TraceContext backed by ActiveSpanSource #257

Closed
vprithvi opened this issue Oct 7, 2017 · 1 comment
Closed

Make TraceContext backed by ActiveSpanSource #257

vprithvi opened this issue Oct 7, 2017 · 1 comment

Comments

@vprithvi
Copy link
Contributor

vprithvi commented Oct 7, 2017

Currently, propagation managed by the tracer through ActiveSpanSource, and propagation managed by TraceContext can diverge, leading to issues like #204

Making the TraceContext backed by ActiveSpanSource, or having them share a common propagation mechanism would prevent this from happening.
@vprithvi vprithvi mentioned this issue Oct 7, 2017
Merged
@vprithvi
Copy link
Contributor Author

Fixed with #266

@pavolloffay pavolloffay mentioned this issue May 16, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vprithvi