matrix-org/synapse

Add additional debug logging to MultiWriterIdGenerator.__init__ and get_cache_dict

Open

#9443 aperta il 18 feb 2021

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Python (2196 fork)batch import
O-OccasionalS-TolerableT-Taskgood first issue

Metriche repository

Star
 (11.713 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.)

Guida contributor

Direzione di ricerca
Fare riferimento a `StreamIdGenerator. init ` che utilizza ` load current id` per la registrazione di debug. Trovare `MultiWriterIdGenerator. init ` e `get cache dict` nella codebase di Synapse (probabilmente in `synapse/storage/databases/main/stream.py`). Aggiungere istruzioni `logger.debug` simili allo schema esistente. Assicurarsi che la registrazione includa informazioni di stato rilevanti per facilitare la diagnosi.
Tech stack
python
Dominio
backendobservability
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Python
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80