Skip to content

Non-deterministic doubleEagerLinearRoundTimer #4241

@pinebit

Description

@pinebit

🐞 Bug Report

Description

Revisit doubleEagerLinearRoundTimer.

The current behavior is non-deterministic which is mandatory for QBFT.
This timer persists the state t.firstDeadlines and tries to adjust the resulting timeouts, which we believe is wrong.

Has this worked before in a previous version?

No

🔬 Minimal Reproduction

Check doubleEagerLinearRoundTimer.Timer() implementation.

🔥 Error

It produces non-deterministic intervals.

🌍 Your Environment

NA

Metadata

Metadata

Assignees

No one assigned

    Labels

    protocolProtocol Team tickets

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions