-
Notifications
You must be signed in to change notification settings - Fork 21.1k
core/stateless, eth: add debug_executionWitnessByHash #32216
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
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Guillaume Ballet <3272758+gballet@users.noreply.github.com>
Signed-off-by: Guillaume Ballet <3272758+gballet@users.noreply.github.com>
Signed-off-by: Guillaume Ballet <3272758+gballet@users.noreply.github.com>
Signed-off-by: Guillaume Ballet <3272758+gballet@users.noreply.github.com>
I checked the difference in witness between geth and reth for block https://etherscan.io/block/23174550. Apart from the differences noted above:
In addition, good to clarify these in the spec:
|
Does anyone know what Keys refers to? |
|
For the same block I realized that the are differences also in the trie nodes. Namely reth and geth share 8536 nodes in common, but reth has an extra 52 not present in geth response, and geth has 4 nodes not present in reth response. The list of these nodes is available on request. One of those nodes that reth includes extra is |
yeah could you please share them? |
Here you go: |
This is a request by @kevaundray.
This PR adds a new system call, which re-executes a block with stateless mode activated, so that the witness data are collected and returned.
This has some differences with the reth equivalent (list will likely :
nil
codes aren't addedkeys
field is present but is always empty - this is an agreement with Kev, that will be done in a later PR. The role of this PR is to find the disagreements with reth.Note that, besides these issues, I still do find some differences with what reth returns, which is why this PR is in Draft mode.