diff options
author | MarcoFalke <falke.marco@gmail.com> | 2021-05-25 08:42:23 +0200 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2021-05-25 08:42:30 +0200 |
commit | aeecb1c2eb57bcdf0f1be4682a5d0b79b5470e57 (patch) | |
tree | 6fe321573a385fff0ea897865fe9047eeee99aaf /doc/descriptors.md | |
parent | 0a909073dc6c9090179b8adedacce288d619d255 (diff) | |
parent | a7a43e8fe85f6247c35d7ff99f36448574f3e34a (diff) | |
download | bitcoin-aeecb1c2eb57bcdf0f1be4682a5d0b79b5470e57.tar.xz |
Merge bitcoin/bitcoin#21992: p2p: Remove -feefilter option
a7a43e8fe85f6247c35d7ff99f36448574f3e34a Factor feefilter logic out (amadeuszpawlik)
c0385f10a133d5d8a4c296e7b7a6d75c9c4eec12 Remove -feefilter option (amadeuszpawlik)
Pull request description:
net: Remove -feefilter option, as it is debug only and isn't used in any tests. Checking this option for every peer on every iteration of the message handler is unnecessary, as described in #21545.
refactor: Move feefilter logic out into a separate `MaybeSendFeefilter(...)` function to improve readability of the already long `SendMessages(...)`. fixes #21545
The configuration option `-feefilter` has been added in 9e072a6e66efbda7d39bf61eded21d2b324323be: _"Implement "feefilter" P2P message"_
According to the [BIP133](https://github.com/bitcoin/bips/blob/master/bip-0133.mediawiki), turning the fee filter off was ment for:
> [...] a node [...] using prioritisetransaction to accept transactions whose actual fee rates might fall below the node's mempool min fee [in order to] disable the fee filter to make sure it is exposed to all possible txid's
`-feefilter` was subsequently set as debug only in #8150, with the motivation that the help message was too difficult to translate.
ACKs for top commit:
jnewbery:
Code review ACK a7a43e8fe85f6247c35d7ff99f36448574f3e34a
promag:
Code review ACK a7a43e8fe85f6247c35d7ff99f36448574f3e34a.
MarcoFalke:
review ACK a7a43e8fe85f6247c35d7ff99f36448574f3e34a 🦁
Tree-SHA512: 8ef9a2f255597c0279d3047dcc968fd30fb7402e981b69206d08eed452c705ed568c24e646e98d06eac118eddd09205b584f45611d1c874abf38f48b08b67630
Diffstat (limited to 'doc/descriptors.md')
0 files changed, 0 insertions, 0 deletions