hbbft icon indicating copy to clipboard operation
hbbft copied to clipboard

Move some `agreement` logic into submodules.

Open afck opened this issue 6 years ago • 2 comments

The Agreement implementation is pretty convoluted, even though some parts of it could be split out and have a somewhat well-defined contract by themselves. E.g. the BVal round is called "Binary-value Broadcast", and together with Aux "Synchronized Binary-value Broadcast" in https://hal.archives-ouvertes.fr/hal-01176110/document.

Maybe we should also factor out AgreementEpoch, to split the epoch management and dispatching from the main part of the logic, and to avoid clearing all those fields individually. (That makes it easy to forget one.)

afck avatar Jul 30 '18 15:07 afck

I'd love to see this API refined and clarified :)

c0gent avatar Jul 30 '18 21:07 c0gent

Unassigning but not closing this for now. It would be good to make a decision about #198 before we continue with refactoring this module.

afck avatar Aug 09 '18 10:08 afck