-
Notifications
You must be signed in to change notification settings - Fork 893
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.
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
Clarify that enabled applies to synchronous instruments #4211
Clarify that enabled applies to synchronous instruments #4211
Conversation
The Enabled API is only to be applied to synchronous instruments. Fix open-telemetry#4200 Signed-off-by: Alex Boten <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Trask Stalnaker <[email protected]>
I'm not sure that it's called out anywhere, this PR addresses:
|
This sounds like SDK implementation bugs/improvements to me rather than a spec issue. |
Anything else needed for this one or can it be merged? |
Fix #4200
Changes
The Enabled API is only to be applied to synchronous instruments.
For non-trivial changes, follow the change proposal process.
CHANGELOG.md
file updated for non-trivial changesspec-compliance-matrix.md
updated if necessary