Skip to content

Conversation

@piotrm50
Copy link
Contributor

@piotrm50 piotrm50 commented Dec 3, 2025

Description of change

Fix differences between Starfish and Mysticeti in handling of previously processed commits at startup by introducing a replay mechanism. Adjust StarfishConsensusHandler to manage commit ranges effectively and log replay progress.

Links to any relevant issues

Fixes #9484

How the change has been tested

  • Basic tests (linting, compilation, formatting, unit/integration tests)
  • Patch-specific tests (correctness, functionality coverage)
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that new and existing unit tests pass locally with my changes

…Starfish.

Fix differences between Starfish and Mysticeti in handling of previously processed commits at startup by introducing a replay mechanism. Adjust `StarfishConsensusHandler` to manage commit ranges effectively and log replay progress.
@piotrm50 piotrm50 self-assigned this Dec 3, 2025
@piotrm50 piotrm50 added the consensus Issues related to the Core Consensus team label Dec 3, 2025
@vercel
Copy link

vercel bot commented Dec 3, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

6 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
apps-backend Ignored Ignored Preview Dec 4, 2025 2:54pm
apps-ui-kit Ignored Ignored Preview Dec 4, 2025 2:54pm
iota-evm-bridge Ignored Ignored Preview Dec 4, 2025 2:54pm
iota-multisig-toolkit Ignored Ignored Preview Dec 4, 2025 2:54pm
rebased-explorer Ignored Ignored Preview Dec 4, 2025 2:54pm
wallet-dashboard Ignored Ignored Preview Dec 4, 2025 2:54pm

@piotrm50 piotrm50 marked this pull request as ready for review December 5, 2025 08:54
@piotrm50 piotrm50 requested review from a team as code owners December 5, 2025 08:54
@piotrm50 piotrm50 force-pushed the consensus/fix/consensushandler-differences branch from 1229c15 to 5a8e091 Compare December 5, 2025 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

consensus Issues related to the Core Consensus team core-protocol

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Check that Starfish and Mysticeti adapter/handler have the same functionality

4 participants