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

Refactor CAT pool to become nonce aware #1498

Open
1 task
Tracked by #1509
ninabarbakadze opened this issue Sep 23, 2024 · 3 comments
Open
1 task
Tracked by #1509

Refactor CAT pool to become nonce aware #1498

ninabarbakadze opened this issue Sep 23, 2024 · 3 comments
Assignees

Comments

@ninabarbakadze
Copy link
Member

ninabarbakadze commented Sep 23, 2024

Description

Currently, transactions are prioritised based on the fee not taking nonce order into account. We can make the mempool nonce aware where transactions would be partially ordered by both nonce and priority, ensuring high-priority transactions are processed sooner while maintaining nonce order. This would allow users to:

  • Cancel transactions
  • Replace transactions with the same nonce and higher fee

Acceptance Criteria

  • CAT pool is nonce aware
@rootulp
Copy link
Collaborator

rootulp commented Sep 24, 2024

CAT pool isn't the default yet and AFAIK we haven't recommended its use on mainnet. Have we considered adding this feature to the other mempools (especially the prioritized mempool)?

@ninabarbakadze
Copy link
Member Author

IIRC @evan-forbes was talking about how CAT will be the only supported mempool soon. Shipping this feature will most likely align with shipping CAT

@rootulp
Copy link
Collaborator

rootulp commented Oct 1, 2024

Thanks for the context. If we aren't going to add this feature to the other mempools, IMO we should work on a few other issues before this one:

  1. Recommend CAT mempool on testnets + mainnet
  2. Deprecate other mempools (needs discussion)
  3. Lastly, this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants