Provide skylight integration #1169
Replies: 8 comments
-
Beta Was this translation helpful? Give feedback.
-
|
Beta Was this translation helpful? Give feedback.
-
class Dispatcher < ::Skylight::Core::Normalizers::Normalizer
register "call.dispatcher.rails_event_store"
CAT = "app.rails_event_store.dispatcher".freeze
def normalize(_trace, _name, payload)
[CAT, "#{payload[:subscriber]}#call", "Handle #{payload[:event].type}"]
end
end |
Beta Was this translation helpful? Give feedback.
-
@pawelpacana this looks good! If you're interested, we can integrate this into the main skylight gem. |
Beta Was this translation helpful? Give feedback.
-
@wagenet thanks! Integrating into skylight gem would be probably the best from the developer perspective. Btw. is there any way to format the description? Like multiple lines for example? |
Beta Was this translation helpful? Give feedback.
-
Right now we don't have a way to do it. You can file an issue at https://github.com/skylightio/skylight-ruby/issues/new and explain a bit more about what you'd like. |
Beta Was this translation helpful? Give feedback.
-
We already have instrumented most of the interesting internals via https://railseventstore.org/docs/instrumentation/ which for
rails_event_store
gem works withActiveSupport::Notifications
.It may be possible that no further work in RES (nor in additional library) is needed. Instead we may want to document the integration process.
https://www.skylight.io/support/getting-more-from-skylight#probes
Beta Was this translation helpful? Give feedback.
All reactions