celestiaorg / celestia-app

Celestia consensus node
https://celestiaorg.github.io/celestia-app/
Apache License 2.0
345 stars 292 forks source link

Increase mempool `MaxTxsBytes` default to 1 GiB #3966

Open rootulp opened 1 month ago

rootulp commented 1 month ago

Context

Per sync discussions elsewhere I think we can increase this to 1 GB

_Originally posted by @evan-forbes in https://github.com/celestiaorg/celestia-app/pull/3961#discussion_r1796043232_

Proposal

  1. Research the default mempool MaxTxBytes for CometBFT
  2. Research the default mempool MaxTxBytes is on other production chains (Cosmos Hub, Osmosis, DYDX, Noble)
  3. Research what are the downsides to increasing/decreasing the mempool MaxTxBytes
  4. If not issues discovered during research, increase the default mempool MaxTxsBytes to 1 GiB