electric icon indicating copy to clipboard operation
electric copied to clipboard

Improve telemetry wrapper clarity

Open msfstef opened this issue 4 months ago • 0 comments

(quoting @alco, followup from https://github.com/electric-sql/electric/pull/3110)

We are currently wrapping :telemetry.execute() calls inside Electric.Telemetry.OpenTelemetry which obscures the fact that those metrics are used not only by our OpenTelemetry exporter but also the Prometheus and CallHomeReporter ones. At the same time the name get_stack_span_attrs() doesn't reflect the fact that those attrs are also put on metrics, not only spans.

We should refactor some of the telemetry to reflect this fact.

msfstef avatar Sep 08 '25 15:09 msfstef