scoutapp / scout_apm_elixir

ScoutAPM Elixir Agent. Supports Phoenix and other frameworks.
https://scoutapm.com
Other
36 stars 20 forks source link

Don't split module names in Ecto instruments #77

Closed cschneid closed 5 years ago

cschneid commented 5 years ago

This now creates a 2 element list that looks like: [:"myapp/repo", :query] as the Telemetry handler mapping.

Fixes #76

cschneid commented 5 years ago

After reading docs some more, this is incorrect. We need to have the repo naming the way it was.

https://github.com/elixir-ecto/ecto/blob/454bd594637c0b45ed9dc7bb66702863236620be/lib/ecto/repo.ex#L92-L118