-
Notifications
You must be signed in to change notification settings - Fork 792
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
[api-metrics]: Create or verify compliance of Counter #2548
Comments
I didn't find any statements on
|
The "optional context" parameter in the issue description came from exactly that line. Of course you can always get context implicitly but as a general rule we also allow explicit context anywhere implicit context is allowed. |
The unused metrics options |
IIUC, the attributes of the second parameter of |
Yes it is. Not sure what I was thinking :)
Agree. Added to the issue description. |
https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/api.md#counter
Current Status 10/27
Currently we have bound and unbound versions. The unbound version satisfies the value and attribute parameters. In #2559 @legendecas has made the unbound version the only version. The following still needs to be done:
The text was updated successfully, but these errors were encountered: