Skip to content
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

Define Metric SDK behavior when presented with invalid metric recording values #1873

Closed
jkwatson opened this issue Aug 17, 2021 · 1 comment
Assignees
Labels
area:sdk Related to the SDK bug Something isn't working spec:metrics Related to the specification/metrics directory

Comments

@jkwatson
Copy link
Contributor

  1. We know that the metrics API cannot throw exceptions/panic on bad inputs.
  2. The Counter (for example) cannot accept negative values to its add operation. Histograms should only accept positive values for recordings.
  3. How should SDK implementations behave when presented with syntactically valid but semantically invalid inputs such as this for metric recordings?
@jkwatson jkwatson added bug Something isn't working spec:metrics Related to the specification/metrics directory labels Aug 17, 2021
@reyang reyang added the area:sdk Related to the SDK label Sep 8, 2021
@reyang reyang self-assigned this Sep 8, 2021
@reyang reyang added this to the Metrics API/SDK Feature Freeze milestone Sep 8, 2021
@reyang
Copy link
Member

reyang commented Oct 15, 2021

Covered by #2007.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:sdk Related to the SDK bug Something isn't working spec:metrics Related to the specification/metrics directory
Projects
None yet
Development

No branches or pull requests

3 participants