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

Revise the view change method #2133

Closed
s8sato opened this issue Apr 21, 2022 · 1 comment
Closed

Revise the view change method #2133

s8sato opened this issue Apr 21, 2022 · 1 comment
Assignees
Labels
Consensus This issue is related to the Sumeragi consensus iroha2-dev The re-implementation of a BFT hyperledger in RUST Optimization Something isn't working as well as it should

Comments

@s8sato
Copy link
Contributor

s8sato commented Apr 21, 2022

Besides shift & shuffle? See discussion

@s8sato s8sato added the iroha2-dev The re-implementation of a BFT hyperledger in RUST label Apr 21, 2022
@s8sato s8sato changed the title Revise the validator election method Revise the view change method Apr 21, 2022
@s8sato
Copy link
Contributor Author

s8sato commented Apr 22, 2022

I'd suggest:

  • Shuffle on the nth view change, where n % 3 == 0
  • Shift by f, otherwise

The idea is to maximize the difference of the set of peers.

This change

  • affects how BlockSynchronizer reproduces topologies EDIT: the impact stays within network_topology::Builder::build
  • might be just a makeshift until the emergence of Hijiri Reputation System #1211

@appetrosyan appetrosyan added Bug Something isn't working Optimization Something isn't working as well as it should labels Jun 6, 2022
@appetrosyan appetrosyan removed the Bug Something isn't working label Jul 6, 2022
@SamHSmith SamHSmith added the Blue label Jul 28, 2022
@appetrosyan appetrosyan removed the Blue label Sep 12, 2022
@SamHSmith SamHSmith added the Consensus This issue is related to the Sumeragi consensus label Feb 22, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Feb 22, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Feb 22, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Feb 22, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Feb 23, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Mar 6, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Mar 6, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Mar 6, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Mar 6, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Mar 6, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Mar 8, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Mar 8, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Mar 9, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Mar 9, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Mar 9, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Mar 9, 2023
SamHSmith added a commit to SamHSmith/iroha that referenced this issue Mar 9, 2023
appetrosyan pushed a commit that referenced this issue Mar 9, 2023
appetrosyan pushed a commit that referenced this issue Jun 5, 2023
mversic pushed a commit that referenced this issue Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Consensus This issue is related to the Sumeragi consensus iroha2-dev The re-implementation of a BFT hyperledger in RUST Optimization Something isn't working as well as it should
Projects
None yet
Development

No branches or pull requests

3 participants