-
Notifications
You must be signed in to change notification settings - Fork 767
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
Consolidate Observer and Counter example #868
Comments
The "Observer example" linked above is a 404; it may of been moved but I'm pretty sure it now refers to https://github.com/open-telemetry/opentelemetry-js/blob/main/experimental/packages/opentelemetry-sdk-metrics-base/examples/metrics/metrics/observer.js. |
The observer API is still experimental. It is likely going to be improved before the GA of the metrics SDK. I'll keep track of this at #2574 |
The observer example was removed in the rewrite. I'm closing this since I feel it is covered by #3026 |
I think we should consolidate
Observer
andCounter
example into one. It would be easier to find and comprehend to the end users.Observer example: https://github.com/open-telemetry/opentelemetry-js/blob/master/examples/metrics/metrics/observer.js
Counter example: https://github.com/open-telemetry/opentelemetry-js/blob/master/examples/prometheus/index.js
The text was updated successfully, but these errors were encountered: