Conversation
| manifest.PrepareProposalDelayMS = 200 | ||
| manifest.ProcessProposalDelayMS = 200 |
There was a problem hiding this comment.
In the "large" case, I would assume PrepareProposal and ProcessProposal are doing "immediate execution", so I'd set the delay to something similar to FinalizeBlock.
Just an idea, up to you.
There was a problem hiding this comment.
Aren't we explicitly not supporting immediate execution? We can still extend these out a little. They're so far quite arbitrary values
There was a problem hiding this comment.
By "immediate execution", I'm referring to the fact that they (PrepareProposal/ProcessProposal) fully execute the block, without waiting for FinalizeBlock to do it.
(N.B.: Even if they execute the block immediately, some of the the block's header fields, like app_hash, are still referring to the previous block, which is not supported in v0.36)
sergio-mena
left a comment
There was a problem hiding this comment.
Thanks for this. Short PR, but valuable
Closes: #7797
This PR adds 5 configrable delays to the e2e application:
And includes these in the generator. By varying the time that these calls take we can better simulate the computational load that real applications experience