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

Set default mempool to CAT #4280

Open
9 tasks
rootulp opened this issue Jan 30, 2025 · 1 comment
Open
9 tasks

Set default mempool to CAT #4280

rootulp opened this issue Jan 30, 2025 · 1 comment
Labels
WS: Big Blonks 🔭 Improving consensus critical gossiping protocols WS: Maintenance 🔧 includes bugs, refactors, flakes, and tech debt etc

Comments

@rootulp
Copy link
Collaborator

rootulp commented Jan 30, 2025

Context

The v1 mempool is currently the default.

Problem

CometBFT v1 removed the v1 mempool. Binary Builders is helping us upgrade to Cosmos SDK v0.52 and CometBFT v1 which is planned to be included in celestia-app v4. Therefore celestia-app v4 may not have the v1 mempool. We should de-couple the roll-out of CAT mempool from celestia-app v4. Ideally while we're still on celestia-app v3, we make the CAT mempool the default.

Proposal

Before celestia-app v4:

  • [Optional] Provide a convenience make command to enable and disable the CAT mempool (similar to bbr-enable / bbr-disable)
  • Encourage node operators on Arabica to switch to the CAT mempool
  • Let that bake for a week
  • Encourage node operators on Mocha to switch to the CAT mempool
  • Let that bake for a week
  • Encourage node operators on Mainnet to switch to the CAT mempool
  • Let that bake for a week
  • Modify docs to use CAT mempool instead of prioritized mempool
  • Make CAT the default mempool for new operators: chore: update mempool defaults #4284
@rootulp
Copy link
Collaborator Author

rootulp commented Jan 30, 2025

Note an alternative option to the one proposed in this issue is to copy the v1 mempool to Comet BFT v1 and then deploy CAT after we release celestia-app v4.

@evan-forbes evan-forbes added WS: Big Blonks 🔭 Improving consensus critical gossiping protocols WS: Maintenance 🔧 includes bugs, refactors, flakes, and tech debt etc and removed needs:triage labels Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WS: Big Blonks 🔭 Improving consensus critical gossiping protocols WS: Maintenance 🔧 includes bugs, refactors, flakes, and tech debt etc
Projects
None yet
Development

No branches or pull requests

2 participants