paritytech / paritytech/try-runtime-cli
Remove DB operations in async code
オープン
まだ誰も着手していません。
enhancement
- 主要言語
- Rust
- スター
- 25
- フォーク
- 29
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
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
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず try-runtime-cli リポジトリ内の非同期コードパスと、それらが行うデータベース操作を特定し、次にそれらの操作が Tokio スケジューラとどのように相互作用するかを判断します。Issue にはファイルやテストのエントリポイントが示されていないため、まず影響を受けるコードを整理し、スケジューラの安全性を検証する方法を確立します。関連する非同期パスが意図した動作を変更せずにデータベース操作を回避できれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- rust
- 領域
- blockchain, databases
- issue の種類
- リファクタリング
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 25/100