Skip to content

Mempool cluster limits#2063

Draft
ImplOfAnImpl wants to merge 6 commits into
mempool_optimizationsfrom
mempool_cluster_limits
Draft

Mempool cluster limits#2063
ImplOfAnImpl wants to merge 6 commits into
mempool_optimizationsfrom
mempool_cluster_limits

Conversation

@ImplOfAnImpl
Copy link
Copy Markdown
Contributor

Mempool will now reject a tx if adding it would form a cluster with tx count or byte size above the limits.

I also exposed MempoolConfig in the node RPC. At the moment, the wallet only uses it to limit the single tx size in places where chain_config.max_tx_size_for_mempool() was previously used. Note that this single tx size limiting isn't (and wasn't) particularly accurate, but fixing it here doesn't make much sense. IMO the wallet's whole utxo selection mechanism has to be revamped, I added the corresponding TODOs.

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

Successfully merging this pull request may close these issues.

1 participant