Interface of binary injector depends on whether we will have a single of two fields for trace context #11
Labels
spec:context
Related to the specification/context directory
Milestone
Comments
|
I am in favor of a single binary blob for trace context in w3c |
|
I'm in favor of a single binary blob as well. |
|
@carlosalberto please comment on issue in W3C on benefits of a single blob over two blobs. |
SergeyKanzhelev
modified the milestones:
API complete,
API proposal done,
API revision: 07-2019
iredelmeier
added
the
spec:context
|
Since it has been resolved ghat we are going to follow https://github.com/w3c/trace-context-binary for our own interface, I am closing this issue for now. |
|
the issue is still there. Current spec expect that formatter can format context into a single bytes blob where spec is written for multiple fields. |
|
This is covered in #426, which raises several points about the need for more specification about binary propagation. These will be addressed in the next milestone. |
TuckTuckFloof
pushed a commit
to TuckTuckFloof/opentelemetry-specification
that referenced
this issue
Oct 15, 2020
…clang-format (open-telemetry#11) * Build env setup scripts * Remove sqlite for now * Reduce the scope of .clang-format .gitattributes and .gitignore changes * Fix code comment
carlosalberto
pushed a commit
to carlosalberto/opentelemetry-specification
that referenced
this issue
Oct 21, 2024
* Document global initializion * Global initializer requirements * Minor revision * Set status * Rename 0010-global-init.md to text/0005-global-init.md * OTr->OTel
carlosalberto
pushed a commit
to carlosalberto/opentelemetry-specification
that referenced
this issue
Oct 23, 2024
* Document global initializion * Global initializer requirements * Minor revision * Set status * Rename 0010-global-init.md to text/0005-global-init.md * OTr->OTel
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Current implementation of binary context inject/extract expects that entire SpanContext can be encoded in a single field:
https://github.com/bogdandrutu/openconsensus/blob/42f90bf8196cc1e3913510160cdfdc3995a7119d/api/src/main/java/openconsensus/context/propagation/BinaryFormat.java#L26-L33
However current proposal lists
traceparentandtracestateas separate headers. See w3c/trace-context-binary#7 with the proposal to change itThe text was updated successfully, but these errors were encountered: