matrix-org/synapse

Add additional debug logging to MultiWriterIdGenerator.__init__ and get_cache_dict

Aberta

#9.443 aberto em 18 de fev. de 2021

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (0 responsável)Python (2.196 forks)batch import
O-OccasionalS-TolerableT-Taskgood first issue

Métricas do repositório

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

Description

We should add debug logging to MultiWriterIdGenerator similar to that in StreamIdGenerator, per https://github.com/matrix-org/synapse/issues/9268#issuecomment-770190320:

If MultiWriterIdGenerator.__init__ and get_cache_dict included some logger.debug, this would be a lot less confusing to diagonose.

(I mention MultiWriterIdGenerator specifically, because StreamIdGenerator.__init__ does write debug, via _load_current_id, and it is quite helpful, and the inconsistency is extra confusing.)

Guia do colaborador

Direção de pesquisa
Examine a depuração existente em StreamIdGenerator. load current id e aplique um padrão semelhante a MultiWriterIdGenerator. init e get cache dict.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Manutenção
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.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Pythonlogging
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70