-
Notifications
You must be signed in to change notification settings - Fork 850
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document precedence order is multiple context propagators are used #1698
Document precedence order is multiple context propagators are used #1698
Conversation
Signed-off-by: Sergei Malafeev <sergei@malafeev.org>
Codecov Report
@@ Coverage Diff @@
## master #1698 +/- ##
============================================
- Coverage 85.51% 85.47% -0.04%
Complexity 1442 1442
============================================
Files 177 177
Lines 5571 5571
Branches 578 578
============================================
- Hits 4764 4762 -2
- Misses 606 607 +1
- Partials 201 202 +1
Continue to review full report at Codecov.
|
I think this is currently unspecified in the spec, as open-telemetry/opentelemetry-specification#496 is still open. If so, I think we should leave this unspecified in Java too to be able to follow the spec later without breaking documented behavior. |
ping @malafeev |
I'm tempted to state/document here that the order is undefined for the time being. That way at least we are not tied for now ;) Does that sound right? If YES, @malafeev, you can iterate on this PR ;) |
Signed-off-by: Sergei Malafeev <sergei@malafeev.org>
updated |
fix #1271