-
Notifications
You must be signed in to change notification settings - Fork 813
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
Update Metrics SDK to latest Feature-Freeze Spec #2574
Comments
Tried to sum up the internal design of the SDK: https://docs.google.com/document/d/1EluDhj5UanNATS3hR8xx30sv55Se7uYjsGYslvNWHtc/edit#. Please feel free to comment :). |
Overall i'm not familiar with all the new mechanisms of the sdk (specially metrics stream) so i won't help much providing feedback on the implementation but i've made few comments on the wording. I'm hoping to be able to re-visit it when PRs are made to help a little bit more on reviews |
Closed and keep tracking in https://github.com/open-telemetry/opentelemetry-js/milestone/15. |
This issue tracks work needed to meet the SDK specification.
Prototype on exporters:
After GA:
The text was updated successfully, but these errors were encountered: