-
Notifications
You must be signed in to change notification settings - Fork 589
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Proposal to update the mempool 1559 parameters for v22 (#7285)
We have observed the mempool 1559 parameters spiking way higher than they need to, due to a series of issue. - Block's have too low of a gas capacity - Different nodes have very divergent mempools, and don't have Ethereum's incentives for maximizing tx fee - The recheck factor forces the base fee to get at minimum 3x higher than the spam rate. - Wallets use a large multiplier (3x) due to the above issues - Leading to users paying 9x over the spam that needs to get evicted at peaks. The need for the recheck factor was to offer a grace period for txs that become candidates for eviction due to momentary surges, or differences between mempool sizes between validator operators. Its because we are in a FIFO mempool, with no way right now to "preserve" user txs at too low of a fee, that may be valid later. Theres no system for keeping them in the network, and no system for flagging the issue to users. An alternative is to keep a higher recheck factor for just RPC nodes, but not validators, if we see user txs not making it in. (That way they are still "remembered" and kept in the network. Though they may never get re-gossipped) With all nodes ideally getting aligned on one fee market / mempool version in the next release, we can revisit this recheck factor. It was already lowered from its previous `4` to `3` in a prior release that got to a subset of nodes. This PR suggests a further 25% reduction to `2.25`. Maybe it should be a bit higher to remain more conservative though? The issue now is really moreso about supporting user txs from occasional fee spikes, instead of handling inconsitencies between 1559 nodes and non-1559 nodes. This further lowers the reset interval as this has gotten to more nodes.
- Loading branch information
1 parent
d3621eb
commit bdcd251
Showing
3 changed files
with
62 additions
and
15 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters