diff options
author | MacroFake <falke.marco@gmail.com> | 2022-07-21 19:54:15 +0200 |
---|---|---|
committer | MacroFake <falke.marco@gmail.com> | 2022-07-21 19:54:18 +0200 |
commit | b8067cd435059eedb580975afc62c4e7a6f27321 (patch) | |
tree | 5a9eaf6c84dcc68828f9fca36847bb1b4e87576f /src/index/base.h | |
parent | d1e42659bbdd8da170542d8c638242cd94f71a7d (diff) | |
parent | 11780f29e7c3f50fb7717fc98950ece9385d314b (diff) | |
download | bitcoin-b8067cd435059eedb580975afc62c4e7a6f27321.tar.xz |
Merge bitcoin/bitcoin#22485: doc: BaseIndex sync behavior with empty datadir
11780f29e7c3f50fb7717fc98950ece9385d314b doc: BaseIndex sync behavior with empty datadir (James O'Beirne)
Pull request description:
Make a note about a potentially confusing behavior with `BaseIndex::m_synced`;
if the user starts bitcoind with an empty datadir and an index enabled,
BaseIndex will consider itself synced (as a degenerate case). This affects
how indices are built during IBD (relying solely on BlockConnected signals vs.
using ThreadSync()).
ACKs for top commit:
mzumsande:
ACK 11780f29e7c3f50fb7717fc98950ece9385d314b
Tree-SHA512: 0b530379e57c62e05d2ddca7bb8e2c936786fa00678f9eaa1bb3742d957c48f141d46f936734b03f6673d964abc7eb72c1769f1784b9d3563d218e96296b7afd
Diffstat (limited to 'src/index/base.h')
-rw-r--r-- | src/index/base.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/index/base.h b/src/index/base.h index 77c46dd428..5a484377e7 100644 --- a/src/index/base.h +++ b/src/index/base.h @@ -55,6 +55,10 @@ private: /// Whether the index is in sync with the main chain. The flag is flipped /// from false to true once, after which point this starts processing /// ValidationInterface notifications to stay in sync. + /// + /// Note that this will latch to true *immediately* upon startup if + /// `m_chainstate->m_chain` is empty, which will be the case upon startup + /// with an empty datadir if, e.g., `-txindex=1` is specified. std::atomic<bool> m_synced{false}; /// The last block in the chain that the index is in sync with. |