opentelemetry-java
opentelemetry-java copied to clipboard
Check if we should preserve more of the Context in the OpenTracing shim.
Currently when a span is created in SpanBuilderShim.start, we only have the parent OpenTracing spanContext available. This means that we lose the originial full context when calling SpanProcessor.OnStart (not yet implemented with that parameter, but see https://github.com/open-telemetry/opentelemetry-specification/pull/875 and https://github.com/open-telemetry/opentelemetry-java/pull/1611#discussion_r493795647.
CC @carlosalberto
@carlosalberto @Oberon00 any idea if we need this still?
Closing as I believe this is not needed. The opentracing shim spec is stable and says that spancontext and baggage are included in the context.
@carlosalberto feel free to re-open if you feel I've misinterpreted something.