IntersectMBO / IntersectMBO/ouroboros-consensus

Take version into account in nodeBlockFetchSize

オープン
#653 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
technical debt
主要言語
Haskell
スター
67
フォーク
43
平均マージ
5日 13時間
マージ済み PR(30日)
43

説明

Currently, `nodeBlockFetchSize` does not take the `BlockNodeToNodeVersion` (it's not used for `NodeToClient` protocols at the moment) into account when coming up with an estimate for the block size. This means that if for a certain version, the blocks are larger, `nodeBlockFetchSize` would have to overestimate for the other versions, just to make sure that it never underestimates.

Note that eventually `nodeBlockFetchSize` should be removed from `RunNode`.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

nodeBlockFetchSize とその RunNode の呼び出し箇所を特定し、BlockNodeToNodeVersion がブロック取得のサイズ計算パスでどのように表現され、使用されているかを調べます。完了の条件は、最終的に RunNode から nodeBlockFetchSize が削除されることを念頭に置きつつ、推定が該当するバージョンを考慮し、大きなブロックを過小評価しないことです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
haskell
領域
blockchain, distributed-systems
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
38/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。