Rotate BReg authority work classes instead of returning after the first submission
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 68/100
調査の方向性
Start in crates/registry-breg/src/review_store.rs at ReviewWorker::run_one and trace the authority work-class loops and their fixed ordering. Add or update the Postgres test described in the issue, and consider the work done when continuously ready submissions no longer prevent a pending cancellation and due result poll from making progress.
索引モデルが issue の本文から書いたものです。
説明
An automated review of #1189 flagged a fairness gap in the BReg review worker's authority pass. In crates/registry-breg/src/review_store.rs, run_one walks its work classes in a fixed order and returns Ok(true) as soon as one unit of work succeeds: submissions first, then cancellations, then the result feed, then result polls.
Failure scenario: pending submissions stay continuously ready, so every pass ends in the submissions loop and the cancellation and result loops below never run. ReviewWorker already stops application jobs from starving authority work, but sustained submission traffic can still starve accepted or cancelling reviews until their recovery or result-retention windows expire.
Suggested fix:
- Rotate the authority work classes across passes (round-robin or a per-class cursor) instead of always restarting at submissions.
- Add a Postgres test with continuously ready submissions that asserts a pending cancellation and a due result poll still make progress.
Filed from the review of #1189; not merge-blocking.
- 主要言語
- Rust
- スター
- 2
- フォーク
- 0
- 平均マージ
- 7時間 50分
- マージ済み PR（30日）
- 240
環境構築
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
registrystack/registry-stack のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
registrystack/registry-stack#1753 ·
メンテナーはふだん 1 日以内に返信
-
area:casework bug criticality:p3 triage:needs-implementation
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
registrystack/registry-stack#1729 ·
メンテナーはふだん 1 日以内に返信
-
area:breg bug criticality:p3 triage:needs-implementation
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
registrystack/registry-stack#1682 ·
メンテナーはふだん 1 日以内に返信
-
area:casework bug criticality:p3 triage:needs-implementation
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
registrystack/registry-stack#1669 ·
メンテナーはふだん 1 日以内に返信
-
agent-ready area:platform criticality:p3 documentation triage:needs-implementation
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
registrystack/registry-stack#1639 ·
メンテナーはふだん 1 日以内に返信
registrystack/registry-stack の issue をすべて見る
似ている issue
-
Change output crossing a compactsize boundary leaves the fee slightly below the requested feerateオープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
bitcoindevkit/bdk_wallet#578 ·
メンテナーはふだん 8 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
521xueweihan/HelloGitHub#3832 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
canonical/opentelemetry-collector-operator#409 ·
メンテナーはふだん 1 日以内に返信