WAL ReadAll() truncates the file even upon transient I/O error
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 65/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- go
- Ambito
- distributed-systems
Direzione di ricerca
Read the WAL's ReadAll() method and the readRecord call shown in the issue. Trace how readRecord reports transient I/O errors versus CRC corruption, then check the WAL tests for coverage of both cases. Done means transient read errors cause the process to fail without truncating the file, while corrupted records retain their intended handling.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
In the WAL's ReadAll() method, we have the following code:
var payloads [][]byte
for bytesToRead > 0 {
payload, bytesRead, err := readRecord(w.file, bytesToRead)
// record was corrupted in wal
if err != nil {
return payloads, w.truncateAt(fileInfo.Size() - bytesToRead)
}
readRecord reads a record and checks its CRC. The readRecord therefore can fail due to two reasons:
- A transient read error
- A corruption of the records inside the file.
The handling in ReadAll doesn't distinguish between the two, and truncates the file anyway.
We therefore need to crash upon a transient read error instead of truncating the file.
- Lingua principale
- Go
- Stelle
- 22
- Fork
- 4
- Merge medio
- 2g 4h
- PR unite (30g)
- 43
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 ava-labs/Simplex
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 56/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 1-2 giorni Idoneità per principianti 52/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
I maintainer di solito rispondono entro 1 giorno
-
Verification accepts aux info appends after the history is sufficient, changing the approval digestAperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di ava-labs/Simplex
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
blocked-needs-validation bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100
projectcapsule/capsule#2280 ·
I maintainer di solito rispondono entro 1 giorno
-
bug ready-for-agent
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
kerti/balances-v2#801 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
[BUG] AdvancedCronJob validating webhook reports incorrect field paths for concurrencyPolicy and pod templatesForse già presa @bhuvan-somisetty l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
openkruise/kruise#2650 ·
I maintainer di solito rispondono entro 1 giorno