Skip to content
This repository has been archived by the owner on Nov 10, 2022. It is now read-only.

Rename NoopSpan to NonRecordingSpan #41

Closed
carlosalberto opened this issue Apr 21, 2021 · 0 comments · Fixed by #43
Closed

Rename NoopSpan to NonRecordingSpan #41

carlosalberto opened this issue Apr 21, 2021 · 0 comments · Fixed by #43
Assignees
@obecny
Labels
specification

Comments

@carlosalberto
Copy link

This is the recently updated name, instead of NoopSpan or DefaultSpan (alternative option, as per the spec, is to abstract it through a method, i.e. Java exposes it through Span Span.wrap(SpanContext).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@obecny obecny

Labels
specification
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: renaming noop span to non recording span obecny/opentelemetry-js-api
3 participants
@carlosalberto @obecny @dyladan