matrix-org/dendrite

Move to zerolog

Offen

#1.468 geöffnet am 02.10.2020

Auf GitHub ansehenContributor Guide
 (12 Kommentare) (4 Reaktionen) (0 zugewiesene Personen)Go (658 Forks)auto 404
T-TaskX-Performancegood first issue

Repository-Metriken

Stars
 (5.632 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Ersetzen Sie logrus durch zerolog oder zap, stellen Sie sicher, dass alle Logaufrufe aktualisiert werden und die Leistungsverbesserungen realisiert werden.
Tech Stack
go
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70