Skip to content

Commit

Permalink
cmd/flags: update max vote amount per block to 64 (#465)
Browse files Browse the repository at this point in the history
After Tripp, the voter set is extended to all validator candidate, whose
upper bound is 64. However, currently, the maximum number of votes per
block is constrained by the flag votepool.maxcurvoteperblock (of value 22),
it should be updated to 64 in default setting. For the same reason, the
maxFutureVoteAmountPerBlock hardcode is also updated from 50 to 64.
  • Loading branch information
andicrypt authored Jun 11, 2024
1 parent a666cb4 commit 3412993
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion cmd/utils/flags.go
Original file line number Diff line number Diff line change
Expand Up @@ -1020,7 +1020,7 @@ var (
MaxCurVoteAmountPerBlock = &cli.IntFlag{
Name: "votepool.maxcurvoteperblock",
Usage: "The maximum finality vote per current block",
Value: 22,
Value: 64,
Category: flags.FastFinalityCategory,
}

Expand Down
2 changes: 1 addition & 1 deletion core/vote/vote_pool.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ import (
)

const (
maxFutureVoteAmountPerBlock = 50
maxFutureVoteAmountPerBlock = 64
maxFutureVotePerPeer = 25

voteBufferForPut = 256
Expand Down

0 comments on commit 3412993

Please sign in to comment.