Skip to content

dbft: don't extend timer when commit is from a different view#21

Merged
fyrchik merged 1 commit intomasterfrom
dont-extend-timer-when-view-differs
Jan 31, 2020
Merged

dbft: don't extend timer when commit is from a different view#21
fyrchik merged 1 commit intomasterfrom
dont-extend-timer-when-view-differs

Conversation

@roman-khimov
Copy link
Member

It can lead to the node being late to trigger a ChangeView when it's the
chosen one to process RecoveryRequest. Fixes #16.

It can lead to the node being late to trigger a ChangeView when it's the
chosen one to process RecoveryRequest. Fixes #16.
@roman-khimov roman-khimov requested a review from fyrchik January 31, 2020 08:49
@codecov
Copy link

codecov bot commented Jan 31, 2020

Codecov Report

Merging #21 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #21   +/-   ##
=======================================
  Coverage   76.55%   76.55%           
=======================================
  Files          22       22           
  Lines        1267     1267           
=======================================
  Hits          970      970           
  Misses        244      244           
  Partials       53       53           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 02c208d...1e9b551. Read the comment docs.

@fyrchik fyrchik merged commit 57394a1 into master Jan 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OnCommit timer extensions related to recovery lead to large delays before ChangeView

2 participants