-
Notifications
You must be signed in to change notification settings - Fork 782
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
flatCallTracer for prebedrock blocks #218
Comments
I assume that you are on OP-mainnet, because block number 105235063 is start of the OP-mainnet bedrock genesis. Also, your are getting some sane responses for prebedrock(block number below 105235063), so I again assume that l2geth is properly setuped on your side(or you are using public infra). Overall node architecture follows below diagram(Ref): If you send your RPC request, it first arrives to op-geth. If op-geth decides that this request is for prebedrock(block number below 105235063), it routes to l2geth, which stores legacy data and has the ability to trace prebedrock transactions. Based on your observation, op-geth fetched upstream(geth) and implements Some sidenote: l2geth and op-geth have different state transition rules. Thats why there are two different binaries, and two different databases. |
Question if |
That seems possible. I proxied |
@ pcw109550 is this something that Optimism Foundation can consider worth doing? |
Rationale
At ethereum/go-ethereum#26377 this PR Geth introduced
flatCallTracer
which usescallTracer
as a proxy to collect call stack info and at the end formats the output in Parity styleCurrently, if I am correct
flatCallTracer
works only for blocks above105235063
and for older blocks response isWhere older blocks support
callTracer
(which means that they should supportflatCallTracer
as well - this is just a different format)This alignment will help a lot of data pipelines to align approaches add process all blockchains using Parity style traces since block 0.
If I am doing something wrong, feel free to advice how to solve that problem.
The text was updated successfully, but these errors were encountered: