mirror of
https://github.com/bitcoin/bitcoin.git
synced 2026-01-20 07:09:15 +01:00
Merge bitcoin/bitcoin#34160: policy: Remove stale rationale paragraph
337b4a2369Remove stale rationale paragraph (flack) Pull request description: It belonged to the note removed in #33892 ACKs for top commit: instagibbs: ACK337b4a2369Tree-SHA512: 3cb1d3b87aa42ff92130af10ce2c286c0d83cbfdf17096d47b540ffe8e1a9a4727aedb8d477599fbff0002d7e262a6a52549dcccfa38dbe61281c221cf26cae2
This commit is contained in:
@@ -99,14 +99,6 @@ submitted as a package.
|
||||
transaction (i.e. in which a replacement transaction with a higher fee cannot be signed) being
|
||||
rejected from the mempool when transaction volume is high and the mempool minimum feerate rises.
|
||||
|
||||
*Rationale*: Avoid situations in which the mempool contains non-bumped transactions below min relay
|
||||
feerate (which we consider to have pay 0 fees and thus receiving free relay). While package
|
||||
submission would ensure these transactions are bumped at the time of entry, it is not guaranteed
|
||||
that the transaction will always be bumped. For example, a later transaction could replace the
|
||||
fee-bumping child without still bumping the parent. These no-longer-bumped transactions should be
|
||||
removed during a replacement, but we do not have a DoS-resistant way of removing them or enforcing a
|
||||
limit on their quantity. Instead, prevent their entry into the mempool.
|
||||
|
||||
Implementation Note: Transactions within a package are always validated individually first, and
|
||||
package validation is used for the transactions that failed. Since package feerate is only
|
||||
calculated using transactions that are not in the mempool, this implementation detail affects the
|
||||
|
||||
Reference in New Issue
Block a user