graphprotocol / graphprotocol/graph-node

Refactor subgraph runner

Ouverte
#5,499 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

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

Description

Subgraph runner has some code duplication and a lot of if-else-ish code. We should try to improve it by making some concerns more isolated and the error more testable.

This is probably going to require several iterations on different parts but the sooner we start the sooner we finish.

As @lutter pointed out, this last PR added some more code duplication which doesn't help https://github.com/graphprotocol/graph-node/pull/4916#issuecomment-1839289761

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.

Piste de recherche

Commencez par localiser le moteur d’exécution des sous-graphes et examinez la duplication abordée dans PR #4916 et dans le commentaire associé. Identifiez les responsabilités et les chemins d’erreur distincts avant de décider de la forme du refactoring ; le travail est terminé lorsque la duplication et la logique fortement chargée en if-else sont réduites et que le comportement des erreurs est couvert par des tests.

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

Évaluation

Stack technique
rust
Domaine
backend
Type d'issue
Refactorisation
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

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