Remove some unused code from `ChainStore`

Ouverte
#3,042 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
52/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
rust
Domaine
backend, blockchain

Piste de recherche

Recherchez ChainStore.confirm_block_hash et ChainStore.block_hashes_by_block_number, puis examinez la commande de copie qui utilise encore ce dernier. Supprimez les deux méthodes inutilisées et mettez à jour la commande de copie afin qu’elle remonte la chaîne de block_offset étapes ; c’est terminé lorsqu’aucune des deux méthodes ne subsiste et que la commande de copie ne dépend plus du calcul du pointeur de bloc.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
Rust
Étoiles
3.2k
Forks
1.1k
Merge moyen
4 j 1 h
PR mergées (30 j)
1

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de graphprotocol/graph-node

Toutes les issues de graphprotocol/graph-node

Issues similaires

Plus d'issues Rust

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.