docs(cli): a poll cadence slower than the chain's block time - #38
Merged
Conversation
…block time smart-router#323 widened the range from [1,8] to [0.25,8]. At 0.25 the tracker polls once per four block times — 8x below the built-in cadence, and the relief that matters on a fast chain. The old floor of 1 was justified by the claim that below it the avgBlockTime-derived windows start to bind. They do not: the staleness window is max(10 x avgBlockTime, 2s) and does not move with this flag — the flag moves the other side of that comparison. Both common cases stay well inside it; the exposure is the seam where an upstream trips the traffic gate and then goes quiet, refreshed by neither polls nor relays. That seam is a property of the product of this flag and the gate's skip budget, so the router warns per chain at startup instead of refusing to start. Documented verbatim, with what to do about it, so the line is read as guidance rather than a failed boot.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#Closes MAG-2990
Why
smart-router#323 widened
--chain-tracker-poll-divisorfrom[1,8]to[0.25,8]. The reference still saysAllowed [1,8]and presents1— "halves the polling rate" — as the largest relief available. At0.25the tracker polls once per four block times, which is 8x below the built-in cadence and the relief that actually matters on a fast chain.The old floor of
1was justified by the claim that below it the router'savgBlockTime-derived windows start to bind. They don't: the staleness window ismax(10 × avgBlockTime, 2s)and does not move with this flag — the flag moves how long a tip can go unrefreshed, the other side of that comparison.What changed
One file,
docs/reference/cli.md:[0.25,8]and names0.25as the eightfold cut, linking to a new subsection.(maxRelaySkips + 1) × interval— 20 ×avgBlockTimeat0.25. That's a property of the product of this flag and the gate's skip budget, so the router warns once per chain at startup rather than refusing to start. The warning is reproduced verbatim, with the explicit note that it's a line to read rather than a failed boot, and what to do if you see it on a bursty chain.The chart-side change is smart-router-helm-chart#120.
How to verify