Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

Trunk/bft yac #1831

Merged
merged 4 commits into from
Nov 5, 2018
Merged

Trunk/bft yac #1831

merged 4 commits into from
Nov 5, 2018

Conversation

l4l
Copy link
Contributor

@l4l l4l commented Nov 2, 2018

Description of the Change

Introduces BFT Yac

Benefits

Final (?) step toward complete BFT consensus

Possible Drawbacks

None?

Signed-off-by: Kitsu <[email protected]>
- Rework Synchronizer in BFT compatible manner
- Extende SyncEvent with height and kNothing outcome
- Update synchronizer tests
- Fixed accordingly GateObjects (Rejects & Agreement on None)

Signed-off-by: Kitsu <[email protected]>
@l4l l4l added needs-review pr awaits review from maintainers consensus labels Nov 2, 2018
@l4l l4l requested review from kamilsa and lebdron November 2, 2018 11:12
@lebdron lebdron changed the base branch from dev to trunk/bft-os-integration November 5, 2018 07:06
@lebdron
Copy link
Contributor

lebdron commented Nov 5, 2018

Changed base to trunk/bft-os-integration since block loading in VoteOther case is not handled yet.

@l4l l4l removed the needs-review pr awaits review from maintainers label Nov 5, 2018
@l4l l4l merged commit f3845a7 into trunk/bft-os-integration Nov 5, 2018
@l4l l4l deleted the trunk/bft_yac branch November 5, 2018 07:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants