paritytech / paritytech/try-runtime-cli

Remove DB operations in async code

Ouverte
#14 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

enhancement
Langage dominant
Rust
Étoiles
25
Forks
29
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

          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

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 les chemins de code asynchrones et leurs opérations de base de données dans le dépôt try-runtime-cli, puis déterminez comment ces opérations interagissent avec le planificateur Tokio. L’issue ne fournit ni fichier ni point d’entrée de test ; commencez donc par cartographier le code concerné et établissez comment vérifier la sûreté du planificateur. La tâche est terminée lorsque les chemins asynchrones concernés évitent les opérations de base de données sans modifier leur comportement prévu.

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

Évaluation

Stack technique
rust
Domaine
blockchain, databases
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.