-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow speculative trx execution in irreversible mode #1030
Comments
What was the reason pre-Savanna for not executing speculative transactions that Savanna invalidates? |
The state was consider too old, 300+ blocks behind. |
Makes sense, thanks. Even with Savanna, would it not matter that transactions are validated on a state that is two blocks old (or possibly more)? |
You can never speculatively execute on the exact state of the produced block. If the node is only one block away instead of two blocks then that is better, but most use cases it will not make any difference. |
… The distributed transaction test now always tests speculative, irreversible, and head.
…_block >= pending_block since blocks are not processed until they become irreversible
…rxs in irreversible mode when not in savanna
Support speculative trx execution in irreversible mode
With the introduction of Savanna instant finality, there is now no longer a reason to prevent speculative trx execution on irreversible nodes. Allowing speculative execution on irreversible nodes would allow a local node to be used for multiple purposes.
Particularly exchanges may find this useful. It would allow them to run in irreversible mode, but still submit their trxs to the chain without the need to run a separate node or use a public API.
The text was updated successfully, but these errors were encountered: