Skip to content

Conversation

@jtescher
Copy link
Member

This patch moves the B3 and trace context propagators into the api/trace
module and renames the distributed context module to propagation.

It also incorporates the doc changes from open-telemetry/opentelemetry-specification#416

This patch moves the B3 and trace context propagators into the api/trace
module and renames the distributed context module to `propagation`.

It also incorporates the doc changes from open-telemetry/opentelemetry-specification#416
@jtescher jtescher requested a review from a team January 30, 2020 04:18
Copy link
Member

@iredelmeier iredelmeier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

open-telemetry/opentelemetry-specification#426 proposes that we remove binary propagators. How about pulling out the binary propagator code into a separate PR for now? Ignore me!

@jtescher jtescher merged commit af4afe8 into open-telemetry:master Feb 4, 2020
@jtescher jtescher deleted the refactor/distributed-context-to-propagation branch February 4, 2020 21:41
Monkeyanator pushed a commit to Monkeyanator/opentelemetry-rust that referenced this pull request Mar 3, 2020
This patch moves the B3 and trace context propagators into the api/trace
module and renames the distributed context module to `propagation`.

It also incorporates the doc changes from open-telemetry/opentelemetry-specification#416
scottgerring pushed a commit to scottgerring/opentelemetry-rust that referenced this pull request Apr 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants