-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
feat(anvil): add mined transactions to state dump #8411
Conversation
Need help figuring out what is the easiest way to create a new |
It seems this still has a problem with impersonated transactions. |
Problem of impersonated transactions is solved, at the cost of breaking compatibility with previous dump format. |
a1dba58
to
f78df87
Compare
836d790
to
707bff3
Compare
@mattsse ptal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this works, ty!
@zerosnacks not sure what is the contribution workflow here. Can it be merged? Need more review? |
Motivation
Fixes #8213
Closes: #7955
Solution
This PR adds the mined transactions that are already part of the memory storage to the file produced by
--dump-state
and loaded by--load-state
.If transactions are not persisted and loaded, queries for full blocks fail.