googleapis / googleapis/google-cloud-cpp
Storage: Review locking on `ObjectDescriptorImpl::OnResume`
- 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
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