-
Notifications
You must be signed in to change notification settings - Fork 895
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
DistributedContext specification is incomplete #242
Comments
Should this be rectified in a simple PR (that simply follows the Java "reference" implementation) or is an OTEP required? |
|
I assume that was just a side note, since renaming would not change anything about the points I raised 😃 |
@Oberon00 in short, the current API is half baked and we hope to finish baking it this week. :) There's was a lot of discussion last week to hammer out details around DistributedContext. You can find the notes here, under "Context Propagation Discussion": https://docs.google.com/document/d/1rgXZik2_-2fU1KpVi7nze5ygWA8NhaLE8VmD20H4P8M/edit#heading=h.o5slz8bi8org We hope to get some proposals out this week, based on this discussion, which will hopefully provide a cleaner proposal for context propagation in general. |
This work has continued as an OTEP: open-telemetry/oteps#66 |
In Java, all of this is implemented in the
DistributedContextManager
(it seems that Tracer : Span ~ DistributedContextManager : DistributedContext).The text was updated successfully, but these errors were encountered: