Skip to content

Non-deterministic doubleEagerLinearRoundTimer #4241

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
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions