googleapis / googleapis/google-cloud-cpp
Storage: Review locking on `ObjectDescriptorImpl::OnResume`
- Lingua principale
- C++
- Stelle
- 659
- Fork
- 462
- Merge medio
- 1g 2h
- PR unite (30g)
- 89
Descrizione
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.
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia individuando ObjectDescriptorImpl::OnResume e seguendo il modo in cui i nuovi messaggi Read() e Write() acquisiscono il mutex e selezionano uno stream. Esamina il codice circostante per la gestione dei messaggi di storage; il lavoro è completo quando il percorso di resume esegue entrambe le operazioni atomicamente oppure garantisce che entrambe utilizzino lo stesso stream, con test che coprano questo comportamento.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- cpp
- Ambito
- cloud
- Tipo di issue
- Refactoring
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 25/100