igorkasyanchuk/rails_performance

question: why append RailsPerformance::Extensions to LogSubscriber instead of integrating with Rails notifications directly

Aperta

#31 aperta il 3 mag 2021

Vedi su GitHub
 (1 commento) (3 reazioni) (0 assegnatari)Ruby (91 fork)github user discovery
help wanted

Metriche repository

Star
 (1237 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

I installed rails_performance on a rails 6.x application and ran a number of page requests. "Recent Requests" included everything as expected. However, the trace info for every request via TraceReport data section was blank.

My rails application uses rails_semantic_logger, which replaces the default ActionView::LogSubscriber and ActiveRecord::LogSubscriber. This prevented rails_performance from tracking the logsubscriber events.

I was able to resolve this by appending the RailsPerformance::Extensions::Db module to the appropriate RailsSemanticLogger LogSubscriber class.

Since ActiveSupport::LogSubscriber is just a logger specific ActiveSupport::Notifications subscriber, why doesn't RailsPerformance just bypass the LogSubscriber altogether and use ActiveSupport::Notifications directly?

If RailsPerformance can use the Notifications directly, would you be comfortable accepting an outside PR?

Guida contributor

Direzione di ricerca
Studia come funziona ActiveSupport::Notifications in Rails e come RailsPerformance si aggancia attualmente a LogSubscriber. Quindi progetta un'integrazione semplice che si iscriva direttamente agli eventi di notifica rilevanti (es. process action.action controller, sql.active record) e catturi i dati delle prestazioni senza dipendere da LogSubscriber. Considera i casi limite come logger personalizzati. Implementa un proof of concept e testa con rails semantic logger.
Tech stack
ruby
Dominio
backend
Tipo issue
Ricerca
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RubyRails
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65