Skip to content

Commit

Permalink
Small workaround to enable using the DAS light client again
Browse files Browse the repository at this point in the history
See code comment of why this makes sense
  • Loading branch information
liamsi committed Jun 12, 2021
1 parent 09d3c3b commit ee88dce
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion consensus/state.go
Original file line number Diff line number Diff line change
Expand Up @@ -1123,7 +1123,16 @@ func (cs *State) defaultDecideProposal(height int64, round int32) {

// cancel ctx for previous proposal block to ensure block putting/providing does not queues up
if cs.proposalCancel != nil {
cs.proposalCancel()
// FIXME(ismail): below commented out cancel tries to prevent block putting
// and providing no to queue up endlessly.
// But in a real network proposals should have enough time in between and if not queuing up to a
// problematic extent will take a lot of time:
// Even on the Cosmos Hub the largest validator only proposes every 15 blocks.
// With an average block time of roughly 7.5 seconds this means almost
// two minutes between two proposals by the strongest validator.
// In our case block interval times will likely be larger.
//
// cs.proposalCancel()
}
cs.proposalCtx, cs.proposalCancel = context.WithCancel(context.TODO())
go func(ctx context.Context) {
Expand Down

0 comments on commit ee88dce

Please sign in to comment.