-
Notifications
You must be signed in to change notification settings - Fork 55
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
OpenTelemetry: implement a mapping from ZIO Metrics Summary to OTEL Histogram #821
Comments
Hi @grouzen |
I have researched some libraries and find this OpenTelemetryMeterRegistry |
Hi @IvanFinochenko! |
@IvanFinochenko |
@IvanFinochenko, we could do a pairing session in Discord, for example, if you want. |
@grouzen it is a good idea. I need a little more time to dive deeper under hood of micrometer and I'll write you. |
So I also took a look at this and I found following things:
I personally see 2 solutions
Currently in zio-telemetry metrics are added directly to Java API counterparts. Instead of doing that we could utilize MetricProducer to instead pull metrics from inside ZIO-Metrics.
From my understanding OTEL is like a consorcium of companies that does telemetry stuff. If they decided that summary metric is deprecated, should we still have it? |
@andrzejressel Hey! I want to clarify this one. From your reply, I understood you are talking about adding support for OTEL Summary, which is legacy. This particular issue is about mapping ZIO metrics Summary to OTEL Histogram because OTEL Summary is deprecated, as stated above. |
This functionality wasn't implemented in #801 as it is tricky to do.
As a reference you can look at the https://github.com/micrometer-metrics/micrometer library. It does exactly this for OTEL integration.
The text was updated successfully, but these errors were encountered: