Skip to content

statement-store: bound recent statement propagation#12666

Open
DenzelPenzel wants to merge 2 commits into
masterfrom
denzelpenzel/bound-recent-statement-propagation
Open

statement-store: bound recent statement propagation#12666
DenzelPenzel wants to merge 2 commits into
masterfrom
denzelpenzel/bound-recent-statement-propagation

Conversation

@DenzelPenzel

@DenzelPenzel DenzelPenzel commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • limit periodic propagation to 256 recent statements per cycle
  • extend StatementStore::take_recent_statements with a maximum batch size while retaining overflow
  • preserve FIFO delivery under sustained submissions with an insertion-ordered recent index
  • keep duplicate resubmissions from moving pending statements to the back of the queue

Context

This extracts the recent-statement propagation optimization from #10941 into a focused change. It is separate from the non-blocking outbound send work in #12657.

Previously, every propagation cycle drained the complete recent-statement index. A large burst could therefore create an unbounded amount of per-cycle filtering, encoding, and sending work. This change bounds that work while ensuring retained statements are propagated in subsequent cycles without starvation.

@DenzelPenzel DenzelPenzel added the T0-node This PR/Issue is related to the topic “node”. label Jul 16, 2026
@paritytech-workflow-stopper

Copy link
Copy Markdown

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/29488124217
Failed job name: fmt

@DenzelPenzel
DenzelPenzel marked this pull request as draft July 16, 2026 09:50
@DenzelPenzel
DenzelPenzel marked this pull request as ready for review July 16, 2026 09:51
@DenzelPenzel

Copy link
Copy Markdown
Contributor Author

/cmd fmt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T0-node This PR/Issue is related to the topic “node”.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant