dashbitco / broadway

Concurrent and multi-stage data ingestion and data processing with Elixir

Home Page:https://elixir-broadway.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Exposing the `:context` value to emitted Telemetry events

CaptainCuddleCube opened this issue · comments

It would be really useful to have the :context value available to Telemetry events.

This doesn't seem like it would be a big change, but it would allow us to have a pipeline name for emitted metrics, which can be scrapped by prometheus!

I'm happy to open a PR to fix this, but before I do, let me know if this is something you guys would consider adding?

Sure, PR is welcome!

I'm even looking at replacing a lot of the telemetry with actual calls to :telemetry.span. It would be nice to have the shared API.