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

R4R: Terminate Update Bonded Validators Iteration Properly #2023

Merged
merged 2 commits into from
Aug 15, 2018

Conversation

alexanderbez
Copy link
Contributor

@alexanderbez alexanderbez commented Aug 14, 2018

Terminate iteration loop in UpdateBondedValidators and UpdateBondedValidatorsFull when the first revoked validator is encountered and perform a sanity check.

closes: #1974


  • Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
  • Updated all relevant documentation (docs/)
  • Updated all relevant code comments
  • Wrote tests
  • Added entries in PENDING.md that include links to the relevant issue or PR that most accurately describes the change.
  • Updated cmd/gaia and examples/

For Admin Use:

  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@alexanderbez alexanderbez changed the title Bez/1974 improve val update iter R4R: Terminate Update Bonded Validators Iteration Properly Aug 14, 2018
@codecov
Copy link

codecov bot commented Aug 14, 2018

Codecov Report

Merging #2023 into develop will increase coverage by <.01%.
The diff coverage is 50%.

@@             Coverage Diff             @@
##           develop    #2023      +/-   ##
===========================================
+ Coverage    64.86%   64.86%   +<.01%     
===========================================
  Files          115      115              
  Lines         6862     6863       +1     
===========================================
+ Hits          4451     4452       +1     
  Misses        2127     2127              
  Partials       284      284

@alexanderbez alexanderbez force-pushed the bez/1974-improve-val-update-iter branch from f2f37f9 to 039b52d Compare August 14, 2018 13:10
Copy link
Contributor

@cwgoes cwgoes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@cwgoes
Copy link
Contributor

cwgoes commented Aug 15, 2018

Please don't squash each time, we can do that once when we merge to develop. I guess the PR checkboxes are confusing.

@alexanderbez
Copy link
Contributor Author

alexanderbez commented Aug 15, 2018

@cwgoes you mean squash right after approvals but right before merge? If so, who does the squashing? Indeed, it's a bit confusing, but I agree nonetheless -- apologies.

@cwgoes
Copy link
Contributor

cwgoes commented Aug 15, 2018

@cwgoes you mean squash right after approvals but right before merge? If so, who does the squashing? Indeed, it's a bit confusing, but I agree nonetheless -- apologies.

The person who merges does the squashing; it's a button on the GitHub UI (or easy enough from a shell too).

Copy link
Contributor

@rigelrozanski rigelrozanski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reviewed - logic lookin' gud~!

@rigelrozanski rigelrozanski merged commit 97ea51a into develop Aug 15, 2018
@rigelrozanski rigelrozanski deleted the bez/1974-improve-val-update-iter branch August 15, 2018 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:x/staking T: Performance Performance improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Staking: UpdateBondedValidators (both variants) should stop at the first jailed validator
3 participants