drive-abci should update evidence consensus params for state sync

Open shumkov opened this issue 9 months ago • 0 comments

For state sync (more precisely, backfill process), we need to tune consensus params.evidence. These params are controlled by ResponsePrepareProposal.ConsensusParamUpdates .

More precisely, backfill process is implemented so that it is filling all blocks up to ConsensusParams.EvidenceParams.MaxAgeNumBlocks AND ConsensusParams.EvidenceParams.MaxAgeDuration (both must be satisfied to end backfill). This is needed so that nodes can validate Evidence. I know we don't use Evidence, but I don't want to comment out respective code.

The issue is that we have set these variables to some quite big numbers, taking 3 h to backfill.

I would suggest to set it to approx 2 * Epoch time, so:

MaxAgeDuration: 20 days
MaxAgeNumBlocks: 15 000

I think it needs protocol version update, so most likely we'd like to include this change in next release.

Mar 25 '25 07:03 shumkov