apple/foundationdb

Remove the double logging for open trace file failure from the TraceEvent cache based implementation

Open

#2.739 geöffnet am 25. Feb. 2020

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (1.266 Forks)batch import
good first issue

Repository-Metriken

Stars
 (13.912 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 4T 12h) (126 gemergte PRs in 30 T)

Beschreibung

With the new issue reporting mechanism in place (PR #2605), the old TraceEvent cache based implementation specifically for trace file open failure can be removed.

Contributor Guide

Research-Richtung
Das Problem erfordert die Entfernung der doppelten Protokollierung bei Fehlern beim Öffnen der Trace Datei. Der neue Problemberichtsmechanismus wurde in PR #2605 eingeführt, daher kann die alte, auf dem TraceEvent Cache basierende Implementierung entfernt werden. Suchen Sie nach dem Code, der das Öffnen der Trace Datei im TraceEvent Cache behandelt, und entfernen Sie die redundante Protokollierung. Die relevante Datei befindet sich wahrscheinlich in den Trace bezogenen Quelldateien von FoundationDB. Die Maintainer haben eine klare Richtung vorgegeben; es sind keine weiteren Untersuchungen erforderlich.
Tech Stack
cpp
Domain
database
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.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++ knowledgeFamiliarity with FoundationDB trace system
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60