Remove some unused code from `ChainStore`

Abierto
#3,042 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
52/100
Tipo de issue
Refactorización
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
rust
Área
backend, blockchain

Línea de trabajo

Busca ChainStore.confirm_block_hash y ChainStore.block_hashes_by_block_number; después, inspecciona el comando de copia que todavía usa este último. Elimina ambos métodos no utilizados y actualiza el comando de copia para que retroceda block_offset pasos en la cadena; se considera terminado cuando ninguno de los dos métodos permanece y el comando de copia ya no depende del cálculo del puntero de bloque.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

The method ChainStore.confirm_block_hash is not used anymore.

The method ChainStore.block_hashes_by_block_number is only used in the copy command, but it looks like we can get rid of that if we follow the chain back block_offset steps instead of calculating a block pointer.

Both these methods should be removed.

Lenguaje dominante
Rust
Estrellas
3.2k
Forks
1.1k
Merge medio
4 d 1 h
PR fusionados (30 d)
1

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de graphprotocol/graph-node

Todos los issues de graphprotocol/graph-node

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.