You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on May 23, 2023. It is now read-only.
While traceId and spanId were recently added to the SpanContext type, routine properties used including parentSpanId and sampling status were left out.
I can cite code back to at least 2012 that uses these properties as well new and different code such as census which exposes the sampling status.
The impact is that usage of the span context will still remain tracer specific. I'm pretty sure you knew this, but I wanted to explicitly make an issue so folks can know why you decided to leave them out.
PS sampling api is still missing from OT, but we are supposed to interpret SAMPLING_PRIORITY the same as if a sampling api call was made, so I think it is relevant.
The text was updated successfully, but these errors were encountered:
While traceId and spanId were recently added to the
SpanContext
type, routine properties used including parentSpanId and sampling status were left out.I can cite code back to at least 2012 that uses these properties as well new and different code such as census which exposes the sampling status.
The impact is that usage of the span context will still remain tracer specific. I'm pretty sure you knew this, but I wanted to explicitly make an issue so folks can know why you decided to leave them out.
PS sampling api is still missing from OT, but we are supposed to interpret SAMPLING_PRIORITY the same as if a sampling api call was made, so I think it is relevant.
The text was updated successfully, but these errors were encountered: