Store swallows DB errors with `.expect()`, propagate Result through storage layer
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- rust
Línea de trabajo
Comienza en crates/storage/src/store.rs revisando los métodos públicos de Store y las llamadas a StorageBackend que actualmente usan .expect(), y luego rastrea sus llamadores en crates/blockchain/src/store.rs. La tarea estará terminada cuando los errores de DB se devuelvan a través de las capas de storage y blockchain store como valores Result, en lugar de provocar panics.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Rust
- Estrellas
- 82
- Forks
- 28
- Merge medio
- 1 d 20 h
- PR fusionados (30 d)
- 20
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de lambdaclass/ethlambda
-
Handle malformed DB data Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
lambdaclass/ethlambda#529 · 4 comentarios ·
-
Document architectural choices Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
lambdaclass/ethlambda#521 · 2 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
lambdaclass/ethlambda#609 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
lambdaclass/ethlambda#602 ·
-
refactor(blockchain): return block-building phase timings instead of deltaing histogram sums Abiertoperformance
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
lambdaclass/ethlambda#599 · 1 comentario ·
Todos los issues de lambdaclass/ethlambda
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
TheLarkInn/aipm#2413 ·
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
alexgorbatchev/simple-ptt#15 ·
-
tooling
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
todo:ticket
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
taikoxyz/taiko-mono#22168 · 1 comentario ·