Store swallows DB errors with `.expect()`, propagate Result through storage layer
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 48/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- rust
- Ambito
- blockchain, distributed-systems
Direzione di ricerca
Inizia in crates/storage/src/store.rs esaminando i metodi pubblici di Store e le chiamate a StorageBackend che attualmente usano .expect(), quindi risali ai relativi chiamanti in crates/blockchain/src/store.rs. Il lavoro è completato quando gli errori DB vengono restituiti attraverso i livelli di storage e blockchain store come valori Result, invece di causare panic.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
crates/storage/src/store.rs calls StorageBackend methods (begin_read, begin_write, put_batch, commit, etc.) but swallows all returned Results via .expect() at 50+ call sites. Any DB error (disk full, corruption, etc.) panics the node instead of being handled gracefully.
The fix is to make Store's public methods return Result<T, Error> and propagate with ?, then update callers in crates/blockchain/src/store.rs accordingly.
Related: #82
Fixing this is also a prerequisite for #266. Without it, the "too many open files" error panics the node rather than being returned as a recoverable error.
- Lingua principale
- Rust
- Stelle
- 82
- Fork
- 28
- Merge medio
- 1g 20h
- PR unite (30g)
- 20
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di lambdaclass/ethlambda
-
Handle malformed DB data Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
lambdaclass/ethlambda#529 · 4 commenti ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
lambdaclass/ethlambda#521 · 2 commenti ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
lambdaclass/ethlambda#609 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
lambdaclass/ethlambda#602 ·
-
performance
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
lambdaclass/ethlambda#599 · 1 commento ·
Tutte le issue di lambdaclass/ethlambda
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
state:needs triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
zed-industries/zed#64680 · 2 commenti ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
RustPython/RustPython#8802 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
TheLarkInn/aipm#2390 ·