googleapis / googleapis/google-cloud-cpp

Storage: Review locking on `ObjectDescriptorImpl::OnResume`

Offen
#15,105 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
api: storage type: cleanup
Vorherrschende Sprache
C++
Sterne
659
Forks
462
Ø Merge
1 T. 2 Std.
Gemergte PRs (30 T.)
89

Beschreibung

The initial implementation reacquires a mutex between starting a new Read() and a new Write() message. We should probably do these atomically, or at least guarantee we use the same stream.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, ObjectDescriptorImpl::OnResume zu lokalisieren und nachzuverfolgen, wie die neuen Read()- und Write()-Nachrichten den Mutex erwerben und einen Stream auswählen. Überprüfe den umgebenden Code zur Verarbeitung von Storage-Nachrichten; abgeschlossen ist die Aufgabe, wenn der Resume-Pfad entweder beide Operationen atomar ausführt oder garantiert, dass beide denselben Stream verwenden, und Tests dieses Verhalten abdecken.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
cloud
Issue-Typ
Refactoring
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.