You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We had intended to be checking height and hash of best block recorded in postgres, but that is not done.
Right now you can nuke your postgresql database, restart kwil, and it thinks the application is at the consensus/state/block height because your abci/info is persisted. This is incorrect.
In (*AbciApp) Info we need to read the best block height and hash from postgres (i think kwild_voting.state with columns height and hash would be just fine) so we can return that to cometbft so it starts replaying blocks.
The text was updated successfully, but these errors were encountered:
I think we concluded the badger kv for abci is now basically useless, just error prone and redundant, @brennanjl , but I wanted to confirm before @charithabandi or anyone else takes the step of removing it.
We had intended to be checking height and hash of best block recorded in postgres, but that is not done.
Right now you can nuke your postgresql database, restart kwil, and it thinks the application is at the consensus/state/block height because your
abci/info
is persisted. This is incorrect.In
(*AbciApp) Info
we need to read the best block height and hash from postgres (i thinkkwild_voting.state
with columnsheight
andhash
would be just fine) so we can return that to cometbft so it starts replaying blocks.The text was updated successfully, but these errors were encountered: