elasticsearch-elixir icon indicating copy to clipboard operation
elasticsearch-elixir copied to clipboard

Telemetry metadata inclusion

Open vanvoljg opened this issue 3 years ago • 0 comments

Including the start metadata in the stop metadata for :telemetry.span-wrapped Elasticsearch calls. This makes it so reporters don't have to retain requests for an entire span and can use only the :stop event to work with metrics.

It also locks sigaws_otp_24, because that was missing previously.

For context, see this issue in the Telemetry repo: https://github.com/beam-telemetry/telemetry/issues/112#issuecomment-1289521937

Specifcially, I'm adding these changes because it seems like upstream Telemetry doesn't feel like it's the correct choice to do this automatically. This implies that libraries which implement Telemetry must do this on their own--include StartMetadata along with StopMetadata whenever possible--and, hence, this PR.

vanvoljg avatar Oct 24 '22 19:10 vanvoljg