Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Nov 25, 2025

Changelist

Blocktimes were reduced from 1s to ~0.6s which decreased the short term order block window
Increase the ShortBlockWindow to 40 to account for the decreased blocktime.

Also double the mempool size to allow for more orders to live in mempool and change TTL to correspond with shortBlockWindow.

Test Plan

[Describe how this PR was tested (if applicable)]

Author/Reviewer Checklist

  • If this PR has changes that result in a different app state given the same prior state and transaction list, manually add the state-breaking label.
  • If the PR has breaking postgres changes to the indexer add the indexer-postgres-breaking label.
  • If this PR isn't state-breaking but has changes that modify behavior in PrepareProposal or ProcessProposal, manually add the label proposal-breaking.
  • If this PR is one of many that implement a specific feature, manually label them all feature:[feature-name].
  • If you wish to for mergify-bot to automatically create a PR to backport your change to a release branch, manually add the label backport/[branch-name].
  • Manually add any of the following labels: refactor, chore, bug.

Summary by CodeRabbit

  • Chores
    • Increased short-term order validity window (longer acceptance window for placing and cancelling orders).
    • Adjusted replay‑validation and rate‑limit thresholds to match the new short‑term window.
    • Increased transaction pool capacity and extended how long pending transactions are retained.
    • Updated end‑to‑end and rate‑limit tests to reflect the new timing and thresholds.

This is an automatic backport of pull request #3226 done by [Mergify](https://mergify.com).

Signed-off-by: Shrenuj Bansal <[email protected]>
Co-authored-by: Kefan Cao <[email protected]>
(cherry picked from commit adf3ad4)
@mergify mergify bot requested a review from a team as a code owner November 25, 2025 19:50
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 25, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@Kefancao Kefancao merged commit d9d41f2 into release/protocol/v9.5.x Nov 25, 2025
23 of 24 checks passed
@Kefancao Kefancao deleted the mergify/bp/release/protocol/v9.5.x/pr-3226 branch November 25, 2025 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants