Skip to content

R4R: Terminate Update Bonded Validators Iteration Properly#2023

Merged
rigelrozanski merged 2 commits intodevelopfrom
bez/1974-improve-val-update-iter
Aug 15, 2018
Merged

R4R: Terminate Update Bonded Validators Iteration Properly#2023
rigelrozanski merged 2 commits intodevelopfrom
bez/1974-improve-val-update-iter

Conversation

@alexanderbez
Copy link
Copy Markdown
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
Copy Markdown

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
@alexanderbez alexanderbez force-pushed the bez/1974-improve-val-update-iter branch from 039b52d to 8a9a5da Compare August 14, 2018 13:13
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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