HomePhabricator

[mempool] Mark mempool import fails that were found in mempool as 'already…

Description

[mempool] Mark mempool import fails that were found in mempool as 'already there'

Summary:
I was investigating the reasons for failed imports in mempool and noticed that LoadMempool() and pwallet->postInitProcess() (for all wallets) are executed concurrently. The wallet will end up importing transactions that LoadMempool() later tries to import; the latter will fail due to the tx already being in the mempool.

Backport of Bitcoin Core PR11062
https://github.com/bitcoin/bitcoin/pull/11062

Test Plan:

make check-all

Reviewers: Fabien, #bitcoin_abc, deadalnix

Reviewed By: Fabien, #bitcoin_abc

Differential Revision: https://reviews.bitcoinabc.org/D4266

Details

Provenance
Karl-Johan Alm <karljohan-alm@garage.co.jp>Authored on Oct 18 2019, 14:24
fpelliccioniCommitted on Oct 18 2019, 14:24
nakihitoPushed on Oct 19 2019, 02:59
Reviewer
Restricted Project
Differential Revision
D4266: [mempool] Mark mempool import fails that were found in mempool as 'already there'
Parents
rSTAGINGd6f375f2c914: [CMAKE] Propagate requirements for cmake >= 3.12
Branches
Unknown
Tags
Unknown