perf(compile): avoid blocking fs reads in the async compile path for the package-config check
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
package_config_check::collect_warningsを見つけ、非同期のcompile経路からどのように呼び出されているかを確認してください。tokio::fsまたはspawn_blockingを使って、範囲が限定されたファイルシステムのスキャンをexecutor外に移すか、ブロッキングI/Oが許容される理由を文書化するかを判断してください。いずれかの選択肢が動作を変えずに実装され、既存のテストが通れば完了です。
索引モデルが issue の本文から書いたものです。
説明
Follow-up from the #2315 review (Rust reviewer, unposted theme).
package_config_check::collect_warnings runs on the async compile path but uses blocking std::fs (read_dir, metadata, read_to_string) to scan at most nine small files at the repository root.
The cost is negligible today, and compile does other synchronous I/O. Even so, the scan should either use tokio::fs or spawn_blocking, or document why blocking I/O is acceptable here, so the pattern doesn't spread to larger scans.
Acceptance criteria
- Either the scan runs off the async executor, or a comment justifies the bounded blocking I/O.
- No behaviour change, and the existing tests pass.
- 主要言語
- Rust
- スター
- 28
- フォーク
- 8
- 平均マージ
- 4日 7時間
- マージ済み PR(30日)
- 5
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
githubnext/ado-aw のほかの issue
-
bug rust
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
githubnext/ado-aw#2359 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
rust test
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
githubnext/ado-aw#2357 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
docs documentation
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
githubnext/ado-aw#1766 ·
メンテナーはふだん 1 日以内に返信
-
agentic-workflows
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
githubnext/ado-aw#1041 ·
メンテナーはふだん 1 日以内に返信
-
security
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
githubnext/ado-aw#855 ·
メンテナーはふだん 1 日以内に返信
githubnext/ado-aw の issue をすべて見る
似ている issue
-
checkpoint/verify/rebuild_indexes on a thread holding a WriteTransaction deadlock instead of INT-001オープンbug known-issue
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
project-minigraf/minigraf#734 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
メンテナーはふだん 4 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
bitcoindevkit/electrum_streaming_client#57 ·
メンテナーはふだん 2 日以内に返信