-
Notifications
You must be signed in to change notification settings - Fork 340
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
specs: consensus leader selection & fork choice #735
Comments
Actually, wouldn't the fork-choice rule be more nuanced depending on what kind of nodes are trying to "choose forks"? |
For now referencing the Tendermint fork choice rule and leaving it at that is probably sufficient. But you're right that the full specification of this is non-trivial, and I'm not sure Tendermint has a full written specification for its fork choice rule at this time. |
Also see: celestiaorg/celestia-core#321 |
We also need to consider:
|
Can link to Tendermint docs |
Summary
Currently, the specs do not define the concrete Leader selection and the Fork Choice rule:
https://github.com/celestiaorg/celestia-specs/blob/06be92043f8ec39bab4222708c0fb44eb7fbca60/src/specs/consensus.md#fork-choice
IMO, these could be very short paragraphs (one or two sentences) that give an overview and refer to tendermint for the details. If there is anything LL specific to consider here (e.g. any difference that stems from the minority assumption), it should also be mentioned somewhere.
Sub-Tasks
We also need to consider:
partly includes: celestiaorg/celestia-node#1134
The text was updated successfully, but these errors were encountered: