Skip to content

dbft: do not send PrepareRequest immediately after Recovery message#75

Merged
roman-khimov merged 1 commit intomasterfrom
fix-recovery-resp
May 15, 2023
Merged

dbft: do not send PrepareRequest immediately after Recovery message#75
roman-khimov merged 1 commit intomasterfrom
fix-recovery-resp

Conversation

@AnnaShaleva
Copy link
Member

We have timer, and we should follow it. Close #74.

@AnnaShaleva AnnaShaleva marked this pull request as draft April 21, 2023 09:21
@AnnaShaleva
Copy link
Member Author

Draft until the neo-project/neo-modules#799 is resolved.

We have timer, and we should follow it. Close #74.

Signed-off-by: Anna Shaleva <anna@nspcc.ru>
@codecov
Copy link

codecov bot commented Apr 21, 2023

Codecov Report

Merging #75 (5a6d1e5) into master (0108387) will increase coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
+ Coverage   75.05%   75.17%   +0.11%     
==========================================
  Files          24       24              
  Lines        1311     1309       -2     
==========================================
  Hits          984      984              
+ Misses        272      271       -1     
+ Partials       55       54       -1     
Impacted Files Coverage Δ
dbft.go 68.73% <ø> (+0.36%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@AnnaShaleva AnnaShaleva marked this pull request as ready for review May 15, 2023 11:35
@roman-khimov roman-khimov merged commit 25db6ba into master May 15, 2023
@roman-khimov roman-khimov deleted the fix-recovery-resp branch May 15, 2023 11:36
AnnaShaleva added a commit to nspcc-dev/neo-go that referenced this pull request May 16, 2023
Send PrepareRequest on timeout only: nspcc-dev/dbft#75.
AnnaShaleva added a commit to nspcc-dev/neo-go that referenced this pull request May 16, 2023
Send PrepareRequest on timeout only: nspcc-dev/dbft#75.

Signed-off-by: Anna Shaleva <shaleva.ann@nspcc.ru>
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.

RecoveryRequest triggers PrepareRequest from Primary even if the time is not yet came to send it

2 participants