matrix-org/dendrite

Move to zerolog

Ouverte

#1 468 ouverte le 2 oct. 2020

Voir sur GitHubGuide contributeur
 (12 commentaires) (4 réactions) (0 personne assignée)Go (658 forks)auto 404
T-TaskX-Performancegood first issue

Métriques du dépôt

Stars
 (5 632 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

We should move to use https://github.com/uber-go/zap or https://github.com/rs/zerolog instead of logrus because logrus does a loooooot of allocations, far more than the rest of Dendrite. I'd propose we use the sugared one since we're not looking for any specific improvement other than "why is logging consuming so much time/allocs". In terms of numbers:

  • logrus 29501 ns/op
  • zap 1250 ns/op (~24x speedup)

Allocations:

  • logrus 125 allocs/op
  • zap 11 allocs/op

Guide contributeur

Direction de recherche
Remplacez logrus par zerolog ou zap, en veillant à ce que tous les appels de journalisation soient mis à jour et que les améliorations de performances soient réalisées.
Stack technique
go
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Go
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70