graphprotocol / graphprotocol/graph-node
Refactor subgraph runner
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Rust
- Sterne
- 3.2k
- Forks
- 1.1k
- Ø Merge
- 4 T. 1 Std.
- Gemergte PRs (30 T.)
- 1
Beschreibung
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
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, den Subgraph-Runner zu finden und die in PR #4916 und dem dort verlinkten Kommentar besprochene Duplizierung zu prüfen. Identifiziere die unterschiedlichen Zuständigkeiten und Fehlerpfade, bevor du die Form des Refactorings festlegst; abgeschlossen ist die Arbeit, wenn die Duplizierung und die if-else-lastige Logik reduziert sind und das Fehlerverhalten durch Tests abgedeckt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rust
- Bereich
- backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100