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.

Sign up for GitHub

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

Jump to bottom

Document DefaultSpan #956

Closed

jsuereth opened this issue Aug 19, 2021 · 0 comments · Fixed by #959

Labels

spec-compliance

Contributor

jsuereth commented Aug 19, 2021

AFAICT, DefaultSpan solves this portion of the spec:

A SpanContext cannot be set as active in a Context directly, but by wrapping it into a Span. For example, a Propagator performing context extraction may need this.

The class should (at a minimum) have documentation.

jsuereth added the spec-compliance label

jsuereth mentioned this issue

[Process] TC Review of 1.0 Trace #946

Closed

13 tasks

lalitb mentioned this issue

Document DefaultSpan, remove DefaultTracer #959

Merged

3 tasks

ThomsonTan closed this as completed in #959

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment