Skip to content
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

Magi doesn't support sequencing #212

Closed
gregdhill opened this issue Feb 14, 2024 · 1 comment
Closed

Magi doesn't support sequencing #212

gregdhill opened this issue Feb 14, 2024 · 1 comment

Comments

@gregdhill
Copy link

gregdhill commented Feb 14, 2024

I may be off-base here as I'm still trying to understand exactly how OP stack works but it's my understanding that magi is an alternative to op-node which includes sequencing. I can see that they override NoTxPool here which would instrument op-geth via the Engine API to construct a block with the L2 transactions. The batcher I assume would collect those transactions directly and submit those to the L1. From what I can tell magi does not support sequencing, or at least no_tx_pool is always set to true, and it is my understanding that the driver is only advanced when it reads a new batch from the L1 - it does not automatically instruct the execution client to build a new block every two seconds. If someone could offer some insight that would be much appreciated!

@gregdhill
Copy link
Author

I see now that #179 exists 🤦

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

No branches or pull requests

1 participant