cockroachdb / cockroachdb/cockroach

replication: checksums assigned to raft log entries, but never used

Abierto
#93,519 2 comentarios 0 reacciones 0 asignados Ver en GitHub
A-kv-replication C-cleanup C-performance T-kv
Lenguaje dominante
Go
Estrellas
32.5k
Forks
4.1k
Métricas de merge de PR
Métricas de PR pendientes

Descripción

We compute a checksum for each Raft log entry here:
https://github.com/cockroachdb/cockroach/blob/da0dacefba5e13119ff7769f95ab33ecd7386490/pkg/kv/kvserver/logstore/logstore.go#L241

However, we never actually verify that the checksum is valid when reading the Raft log.

We should either use this checksum or stop spending resources (proportional to log entry size) computing it.

Jira issue: CRDB-22377

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.