crypto-com / thaler

Thaler Experimental Network; For Crypto.org Chain: github.com/crypto-org-chain/chain-main
https://thaler-testnet.crypto.com
Other
158 stars 80 forks source link

Problem: It's possible for abci to keep wrong state when consensus connection restart #1829

Open yihuang opened 4 years ago

yihuang commented 4 years ago

With following sequence of events, abci could left with incorrect state, because both tx1 and tx2 will be applied?

Possible solutions:

leejw51crypto commented 4 years ago

abci is deterministic to tendermint, maybe is it guaranteed in tendermint level?

tomtau commented 4 years ago

for the option one -- we can try if the next version of rust abci "quits" if the TM connection is closed: https://github.com/tendermint/rust-abci/pull/132/files#diff-4ce93534efc34e923ce01e975eb7ed80R44