graphprotocol / graphprotocol/graph-node
Remove some unused code from `ChainStore`
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Rust
- Estrellas
- 3.2k
- Forks
- 1.1k
- Merge medio
- 4 d 1 h
- PR fusionados (30 d)
- 1
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.
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.
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.
Evaluación
- Stack tecnológico
- rust
- Área
- backend, blockchain
- Tipo de issue
- Refactorización
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 52/100