HomePhabricator

Call TransactionRemovedFromMempool in the CScheduler thread

Description

Call TransactionRemovedFromMempool in the CScheduler thread

Summary:
This is both good practice (we want to move all such callbacks
into a background thread eventually) and prevents a lock inversion
when we go to use this in wallet (mempool.cs->cs_wallet and
cs_wallet->mempool.cs would otherwise both be used).

This is extracted from Core's PR10286 commit 3ea8b75281edc60078423bd5d277cd2a84aa5d33

Depends on D2263

Test Plan:

make check
./test/functional/test_runner.py

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Subscribers: teamcity, schancel

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

Details

Provenance
Matt Corallo <git@bluematt.me>Authored on Jun 8 2017, 15:05
deadalnixCommitted on Jan 8 2019, 14:29
deadalnixPushed on Jan 8 2019, 14:29
Reviewer
Restricted Project
Differential Revision
D2265: Call TransactionRemovedFromMempool in the CScheduler thread
Parents
rABC8b4b0382782c: Add a CValidationInterface::TransactionRemovedFromMempool
Branches
Unknown
Tags
Unknown