Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EFM] Update Epoch Lookup Component #5763

Open
3 tasks done
jordanschalm opened this issue Apr 23, 2024 · 0 comments
Open
3 tasks done

[EFM] Update Epoch Lookup Component #5763

jordanschalm opened this issue Apr 23, 2024 · 0 comments
Assignees

Comments

@jordanschalm
Copy link
Member

jordanschalm commented Apr 23, 2024

The Epoch Lookup acts as a cache supporting queries for the epoch counter corresponding to a given view.

Changes:

  • Add EpochExtended event handler:
    • Set epochRange.finalView to final view of extension
  • Add EpochRecovered event handler:
    • Add epochRange for RecoveryEpoch (Note: will be similar to handler for EpochCommittedPhaseStarted)
  • Remove/revisit [this sanity check](
    // sanity check: ensure counters/views are sequential
    if epoch.counter != latestCachedEpoch.counter+1 {
    return fmt.Errorf("non-sequential epoch counters: adding epoch %d when latest cached epoch is %d", epoch.counter, latestCachedEpoch.counter)
    }
    ) that counters are sequential

Depends on

  1. S-BFT S-Epochs
    durkmurder
  2. S-BFT S-Epochs
    durkmurder
  3. S-BFT S-Epochs
    durkmurder
@jordanschalm jordanschalm changed the title Epoch Lookup Component [EFM] Update Epoch Lookup Component Apr 23, 2024
@kc1116 kc1116 self-assigned this Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants