igorkasyanchuk/rails_performance

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

Aberta

#31 aberto em 3 de mai. de 2021

Ver no GitHub
 (1 comentário) (3 reações) (0 responsável)Ruby (91 forks)github user discovery
help wanted

Métricas do repositório

Stars
 (1.237 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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?

Guia do colaborador

Direção de pesquisa
Estude como o ActiveSupport::Notifications funciona no Rails e como o RailsPerformance atualmente se conecta ao LogSubscriber. Em seguida, projete uma integração simples que se inscreva diretamente nos eventos de notificação relevantes (por exemplo, process action.action controller, sql.active record) e capture dados de desempenho sem depender do LogSubscriber. Considere casos extremos como loggers personalizados. Implemente uma prova de conceito e teste com rails semantic logger.
Pilha de tecnologia
ruby
Domain
backend
Tipo Issue
Pesquisa
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
RubyRails
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65