-
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
Mark ExplicitBucketBoundaries hint API as stable #3391
Comments
It seems like we should have a few implementations of this API before we stabilize it, right? |
ya, the HTTP semconv stability WG would be very grateful if any other languages besides Java can implement since it is required for HTTP semconv stability |
@reyang will be prototyping in .NET ❤️ we will need to prototype in one more language cc @open-telemetry/python-approvers @open-telemetry/cpp-approvers @open-telemetry/javascript-approvers @open-telemetry/go-approvers 🤞 |
Go might be able to prototype this in a few weeks if we stabilize the metric API first. |
related #3621 |
related #3622 |
JS implementation: open-telemetry/opentelemetry-js#3876 |
JS prototype looks fine to me |
@open-telemetry/dotnet-maintainers have you had a chance to prototype this in .NET? I think we just need one more (we now have Java and JS prototypes). |
I think we need resolutions to the following issues before we can proceed with this: |
Just checking in on this, I think we only have 2 of the 3 required prototypes needed to mark this stable: |
@trask: Go prototype open-telemetry/opentelemetry-go#4341 Thanks @dashpole 🎉 |
A little late but also Erlang/Elixir implementation is in PR open-telemetry/opentelemetry-erlang#628 |
What are you trying to achieve?
Mark ExplicitBucketBoundaries hint API as stable since this will be required for emitting HTTP semantic conventions now that we are changing
http.server.duration
andhttp.client.duration
unit from milliseconds to seconds (see #3390).The text was updated successfully, but these errors were encountered: