Skip to content

Conversation

@JulianGCalderon
Copy link
Contributor

Refactors the execution module to always generate a state_dump, even if the transaction fails to execute.

To do this, I created a new function execute_and_process_tx that wraps execute_tx, and processes the result of the transaction, that is:

  • Create state dump (always).
  • Create libfunc profile.
  • Log the result.

Also, we were returning Result<TransactionExecution>, where TransactionExecution also contains a Result, I unified both results into a single Result<TransactionExecution, where TransactionExecution now doesn't contain Result. This allows us to handle the error case more easily.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants