Skip to content

refactor(budget): share pacing state - #8464

Draft
mediocregopher wants to merge 2 commits into
mediocregopher/budget-stack-1-decisionsfrom
mediocregopher/budget-stack-2-shared
Draft

mediocregopher wants to merge 2 commits into
mediocregopher/budget-stack-1-decisionsfrom
mediocregopher/budget-stack-2-shared

Conversation

@mediocregopher

@mediocregopher mediocregopher commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Shares build and validation feedback between consensus and the payload builder. Keeps the existing multiplier algorithm and fixed consensus budget, and passes the same handle through startup, overrides, and the e2e harness.

Part 2/7 of the stack splitting #7838. Added unit tests from that PR are omitted; existing tests are preserved.

Stack: #8463 → #8464 → #8465 → #8466 → #8467 → #8468 → #8469. Merge in order using merge commits to preserve the stack ancestry.

Four busy-load pairs measured mean TPS 15,249 → 15,021 (−1.5%), with individual changes −7.0%, −0.6%, −2.1%, and +4.2%. The last pair reversed the revision order; the shared-state revision was faster there. Results vary with skipped views and long-tail stalls, and do not show a consistent throughput regression. First run, repeat, reversed order.

At 100 TPS offered load, measured chain TPS was 88 → 90, with pooled block-time p50/p75 637/667 ms → 646/691 ms. Low-load comparison.

@mediocregopher mediocregopher added codex A-block-building Related to block building C-refactor agentic-audit PR created or modified by an AI agent labels Oct 20, 2026
@tempoxyz-bot

tempoxyz-bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

👁️ Cyclops Security Review

Last reviewed: no commit yet

This branch has not been deployed

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

Labels

A-block-building Related to block building agentic-audit PR created or modified by an AI agent C-refactor codex

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants