dashbitco / broadway_dashboard

Keep track of your Broadway pipelines from Phoenix LiveDashboard
https://elixir-broadway.org/
Apache License 2.0
220 stars 17 forks source link

Improve Telemetry performance #23

Closed louisvisser closed 1 year ago

louisvisser commented 1 year ago

Addresses this warning regarding :telemetry.attach/4:

Function passed as a handler with ID {BroadwayDashboard.Telemetry, #PID<0.1535.0>} is local function.
This mean that it is either anonymous function or capture of function without module specified. That may cause performance penalty when calling such handler. For more details see note in `telemetry:attach/4` documentation.

https://hexdocs.pm/telemetry/telemetry.html#attach-4
philss commented 1 year ago

@louisvisser good call. Thank you! :purple_heart: