Remove extraneous LogPrint from fee estimation #9123

pull morcos wants to merge 1 commits into bitcoin:master from morcos:eliminateFeeWarning changing 1 files +6 −9

morcos commented at 7:37 PM on November 10, 2016: member

Once priority estimation was removed, not all transactions in the mempool are tracked in the fee estimation mempool tracking (have an entry in mapMemPoolTxs). So there is no error if a transaction is not found for removal.

This will eliminate these incorrect lines from debug.log: 2016-11-06 07:19:18.724305 Blockpolicy error mempool tx 93100e69a817a6b2034456ad3127124f63b9a89d71704ed11d584d5db5fefc54 not found for removeTx

Remove extraneous LogPrint from fee estimation

Once priority estimation was removed, not all transactions in the mempool are tracked in the fee estimation mempool tracking.  So there is no error if a transaction is not found for removal.

fc69eb5c1d

fanquake added the label Docs and Output on Nov 11, 2016
laanwj commented at 9:53 AM on November 11, 2016: member

utACK fc69eb5
morcos commented at 3:36 PM on November 11, 2016: member

@laanwj This is fine to merge if those errors are annoying, but I'm actually going to refactor slightly to clean up the rest of fee estimation, so we could wait as well...
morcos commented at 9:32 PM on November 11, 2016: member

Included in #9138
morcos closed this on Nov 11, 2016
MarcoFalke locked this on Sep 8, 2021

Contributors

morcos

laanwj

Labels

Docs