Stops empty block production amd sets min period to zero #46
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The reason for this change is because the block sealing process was setting the header.Time timestamp into the future, delaying consensus approximately ~200ms beyond when the transactions were applied to the EVM. This could result in situations where transactions are applied correctly during simulation, but fail once consensus check is complete, because the final header.Time is 200ms into the future of the initial simulation.
This change reduces the latency of the geth transaction processing from time a trasnaction enters the mempool to it's finalization at the consensus layer to about <50ms. This is a significant improvement from the existing latency which was in the order of 200ms - 1000ms.