-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
handshake replay through consensus using mockApp #408
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #408 +/- ##
===========================================
- Coverage 51.14% 50.29% -0.86%
===========================================
Files 37 38 +1
Lines 4452 4464 +12
===========================================
- Hits 2277 2245 -32
- Misses 1959 2022 +63
+ Partials 216 197 -19 Continue to review full report at Codecov.
|
consensus/state.go
Outdated
cs := NewConsensusState(config, state, proxyApp, blockStore, mempool) | ||
|
||
evsw := types.NewEventSwitch() | ||
evsw.Start() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
defer evsw.Stop
?
state/execution.go
Outdated
@@ -273,41 +269,81 @@ func (s *State) CommitStateUpdateMempool(proxyAppConn proxy.AppConnConsensus, bl | |||
return nil | |||
} | |||
|
|||
// apply a nd commit a block, but with out all the state validation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and
without
state/execution.go
Outdated
} | ||
|
||
// h.replayLastBlock(h.config, h.state, proxyApp.Consensus(), h.store) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why leave this comment?
Issue: #352
Related: #388, #347, #409
Mempool
andBlockstore
interfaces. Should we move these totypes
?types.Txs
.cs_wal_dir
->cs_wal_file
.consensus/test_data/build.sh
to allow selecting which data to make.many_blocks.cswal
data for testing handshake replay of many blocks.