paritytech / paritytech/try-runtime-cli

Remove DB operations in async code

Offen
#14 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

enhancement
Vorherrschende Sprache
Rust
Sterne
25
Forks
29
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

          you don't have to do it in this PR but worth investing whether it makes things faster or not.

Ideally, we shouldn't do any DB operations in async code it might fuckup the async scheduler in tokio

Originally posted by @niklasad1 in https://github.com/paritytech/substrate/pull/14057#discussion_r1181570805

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, die asynchronen Codepfade und ihre Datenbankoperationen im Repository try-runtime-cli zu lokalisieren, und ermittle anschließend, wie diese Operationen mit dem Tokio-Scheduler interagieren. Das Issue enthält keinen Datei- oder Testeinstiegspunkt. Ordne daher zuerst den betroffenen Code zu und stelle fest, wie sich die Sicherheit des Schedulers überprüfen lässt. Erledigt ist die Aufgabe, wenn die relevanten asynchronen Pfade Datenbankoperationen vermeiden, ohne ihr vorgesehenes Verhalten zu ändern.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
rust
Bereich
blockchain, databases
Issue-Typ
Refactoring
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.