HotShot icon indicating copy to clipboard operation
HotShot copied to clipboard

[TECH DEBT] - Replace `StakeTableScheme` trait bound with stake table snapshot

Open mrain opened this issue 1 year ago • 0 comments

What is this task and why do we need to work on it?

The current StakeTableScheme trait defines a richer set of interfaces beyond HotShot needs. The consensus only needs to take a stake table snapshot from the sequencer and perform read-only query to it. There's no reason to bother with stake table updates and versioning stuff.

We should replace the StakeTableScheme trait bounds in HotShot components, for example the QuorumCertificateScheme trait, with a StakeTableSnapshot trait or struct.

What work will need to be done to complete this task?

  • [ ] Define a StakeTableSnapshot trait or struct
  • [ ] For other HotShot components, replace the StakeTableScheme trait bound with snapshot.
  • [ ] (Maybe) Move StakeTableScheme to the sequencer repo

Are there any other details to include?

No response

What are the acceptance criteria to close this issue?

as described above

Branch work will be merged to (if not the default branch)

No response

mrain avatar Jan 10 '24 19:01 mrain