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

Update txpool to handle multiple currencies #53

Closed
2 tasks done
Tracked by #6
palango opened this issue Sep 19, 2023 · 1 comment
Closed
2 tasks done
Tracked by #6

Update txpool to handle multiple currencies #53

palango opened this issue Sep 19, 2023 · 1 comment
Assignees

Comments

@palango
Copy link
Collaborator

palango commented Sep 19, 2023

Part of #6

Extend the txpool to be able to handle transaction with different currencies for gas payments.

In the current implementation we added a multi currency heap to improve performance. For me it's unclear if this is worth the additional complexity and increased diff, so we should do some benchmarking around this to get a better understanding.

Tasks

Preview Give feedback
  1. hbandura
@palango palango added this to the Testnet with Celo Features milestone Sep 19, 2023
@palango palango self-assigned this Sep 19, 2023
@palango
Copy link
Collaborator Author

palango commented Sep 20, 2023

It might also make sense to postpone this until ethereum-optimism/op-geth#136 has been merged, as that upstream merge includes the split of the txpool into two sub-pools.

This has been done, I'll merge this into our repo.

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

3 participants