-
Notifications
You must be signed in to change notification settings - Fork 55
LEVM: Separate get_state_transitions
from execute_block
#2504
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
Comments
Yeah, as Juli says. The idea of this is to have |
A follow up for this PR could be implementing |
) **Motivation** Currently during batch processing, the state transitions are calculated for every block and then merged, when it would be more performant to calculate them once at the end. **Description** This PR removes the account updates from the execution result and makes every consumer manually request them. <!-- Link to issues: Resolves #111, Resolves #222 --> Closes #2504
Currently, the
execute_block
function returns theBlockExecutionResult
with the account updates, which are calculated by making a call toget_state_transition
function. This is ok when we want to execute just one block, but if we want to execute blocks in batches we don't really need to get the account updates after each block execution, we could just obtain it at the end instead.We want to separate the block execution from the accounts update calculation.
The text was updated successfully, but these errors were encountered: