Merge #6203: feat: only require reindexing when the index was on going to off

c96f9e0285 feat: only require reindexing when the index was off going to off (pasta)

Pull request description:

  ## What was done?
  It does not seem reasonable that we need to reindex when turning indexes off. this logic was introduced in e0781095f0

  ## How Has This Been Tested?
  Hasn't

  ## Breaking Changes
  None, basically

  ## Checklist:
    _Go over all the following points, and put an `x` in all the boxes that apply._
  - [ ] I have performed a self-review of my own code
  - [ ] I have commented my code, particularly in hard-to-understand areas
  - [ ] I have added or updated relevant unit/integration/functional/e2e tests
  - [ ] I have made corresponding changes to the documentation
  - [x] I have assigned this pull request to a milestone _(for repository code-owners and collaborators only)_

ACKs for top commit:
  knst:
    utACK c96f9e0285

Tree-SHA512: a5b634b9fbb6632b35286055a8e5b8db340738b7327a7d860e645ec1f6ef5b5cddc140a3e1c51b09c2f47db375c83e7c53fe5eaf221ab8df1d173caf50fa862d
This commit is contained in:
pasta 2024-10-24 13:23:19 -05:00
commit ce28029063
No known key found for this signature in database
GPG Key ID: E2F3D7916E722D38
7 changed files with 53 additions and 47 deletions

View File

@ -1952,23 +1952,31 @@ bool AppInitMain(NodeContext& node, interfaces::BlockAndHeaderTipInfo* tip_info)
return InitError(_("Incorrect or no devnet genesis block found. Wrong datadir for devnet specified?")); return InitError(_("Incorrect or no devnet genesis block found. Wrong datadir for devnet specified?"));
} }
if (!fReset && !fReindexChainState) {
// Check for changed -addressindex state // Check for changed -addressindex state
if (fAddressIndex != args.GetBoolArg("-addressindex", DEFAULT_ADDRESSINDEX)) { if (!fAddressIndex && fAddressIndex != args.GetBoolArg("-addressindex", DEFAULT_ADDRESSINDEX)) {
strLoadError = _("You need to rebuild the database using -reindex to change -addressindex"); strLoadError = _("You need to rebuild the database using -reindex to enable -addressindex");
break; break;
} }
// Check for changed -timestampindex state // Check for changed -timestampindex state
if (fTimestampIndex != args.GetBoolArg("-timestampindex", DEFAULT_TIMESTAMPINDEX)) { if (!fTimestampIndex && fTimestampIndex != args.GetBoolArg("-timestampindex", DEFAULT_TIMESTAMPINDEX)) {
strLoadError = _("You need to rebuild the database using -reindex to change -timestampindex"); strLoadError = _("You need to rebuild the database using -reindex to enable -timestampindex");
break; break;
} }
// Check for changed -spentindex state // Check for changed -spentindex state
if (fSpentIndex != args.GetBoolArg("-spentindex", DEFAULT_SPENTINDEX)) { if (!fSpentIndex && fSpentIndex != args.GetBoolArg("-spentindex", DEFAULT_SPENTINDEX)) {
strLoadError = _("You need to rebuild the database using -reindex to change -spentindex"); strLoadError = _("You need to rebuild the database using -reindex to enable -spentindex");
break; break;
} }
}
chainman.InitAdditionalIndexes();
LogPrintf("%s: address index %s\n", __func__, fAddressIndex ? "enabled" : "disabled");
LogPrintf("%s: timestamp index %s\n", __func__, fTimestampIndex ? "enabled" : "disabled");
LogPrintf("%s: spent index %s\n", __func__, fSpentIndex ? "enabled" : "disabled");
// Check for changed -prune state. What we are concerned about is a user who has pruned blocks // Check for changed -prune state. What we are concerned about is a user who has pruned blocks
// in the past, but is now trying to run unpruned. // in the past, but is now trying to run unpruned.

View File

@ -394,16 +394,10 @@ bool BlockManager::LoadBlockIndexDB()
// Check whether we have an address index // Check whether we have an address index
m_block_tree_db->ReadFlag("addressindex", fAddressIndex); m_block_tree_db->ReadFlag("addressindex", fAddressIndex);
LogPrintf("%s: address index %s\n", __func__, fAddressIndex ? "enabled" : "disabled");
// Check whether we have a timestamp index // Check whether we have a timestamp index
m_block_tree_db->ReadFlag("timestampindex", fTimestampIndex); m_block_tree_db->ReadFlag("timestampindex", fTimestampIndex);
LogPrintf("%s: timestamp index %s\n", __func__, fTimestampIndex ? "enabled" : "disabled");
// Check whether we have a spent index // Check whether we have a spent index
m_block_tree_db->ReadFlag("spentindex", fSpentIndex); m_block_tree_db->ReadFlag("spentindex", fSpentIndex);
LogPrintf("%s: spent index %s\n", __func__, fSpentIndex ? "enabled" : "disabled");
return true; return true;
} }

View File

@ -4535,7 +4535,13 @@ bool ChainstateManager::LoadBlockIndex()
// needs_init. // needs_init.
LogPrintf("Initializing databases...\n"); LogPrintf("Initializing databases...\n");
InitAdditionalIndexes();
}
return true;
}
void ChainstateManager::InitAdditionalIndexes()
{
// Use the provided setting for -addressindex in the new database // Use the provided setting for -addressindex in the new database
fAddressIndex = gArgs.GetBoolArg("-addressindex", DEFAULT_ADDRESSINDEX); fAddressIndex = gArgs.GetBoolArg("-addressindex", DEFAULT_ADDRESSINDEX);
m_blockman.m_block_tree_db->WriteFlag("addressindex", fAddressIndex); m_blockman.m_block_tree_db->WriteFlag("addressindex", fAddressIndex);
@ -4547,8 +4553,7 @@ bool ChainstateManager::LoadBlockIndex()
// Use the provided setting for -spentindex in the new database // Use the provided setting for -spentindex in the new database
fSpentIndex = gArgs.GetBoolArg("-spentindex", DEFAULT_SPENTINDEX); fSpentIndex = gArgs.GetBoolArg("-spentindex", DEFAULT_SPENTINDEX);
m_blockman.m_block_tree_db->WriteFlag("spentindex", fSpentIndex); m_blockman.m_block_tree_db->WriteFlag("spentindex", fSpentIndex);
}
return true;
} }
bool CChainState::AddGenesisBlock(const CBlock& block, BlockValidationState& state) bool CChainState::AddGenesisBlock(const CBlock& block, BlockValidationState& state)

View File

@ -1044,6 +1044,8 @@ public:
//! Load the block tree and coins database from disk, initializing state if we're running with -reindex //! Load the block tree and coins database from disk, initializing state if we're running with -reindex
bool LoadBlockIndex() EXCLUSIVE_LOCKS_REQUIRED(cs_main); bool LoadBlockIndex() EXCLUSIVE_LOCKS_REQUIRED(cs_main);
//! Initialize additional indexes and store their flags to disk
void InitAdditionalIndexes() EXCLUSIVE_LOCKS_REQUIRED(cs_main);
//! Check to see if caches are out of balance and if so, call //! Check to see if caches are out of balance and if so, call
//! ResizeCoinsCaches() as needed. //! ResizeCoinsCaches() as needed.

View File

@ -38,14 +38,13 @@ class AddressIndexTest(BitcoinTestFramework):
self.import_deterministic_coinbase_privkeys() self.import_deterministic_coinbase_privkeys()
def run_test(self): def run_test(self):
self.log.info("Test that settings can't be changed without -reindex...") self.log.info("Test that settings can be disabled without -reindex...")
self.stop_node(1) self.restart_node(1, ["-addressindex=0"])
self.nodes[1].assert_start_raises_init_error(["-addressindex=0"], "You need to rebuild the database using -reindex to change -addressindex", match=ErrorMatch.PARTIAL_REGEX)
self.start_node(1, ["-addressindex=0", "-reindex"])
self.connect_nodes(0, 1) self.connect_nodes(0, 1)
self.sync_all() self.sync_all()
self.log.info("Test that settings can't be enabled without -reindex...")
self.stop_node(1) self.stop_node(1)
self.nodes[1].assert_start_raises_init_error(["-addressindex"], "You need to rebuild the database using -reindex to change -addressindex", match=ErrorMatch.PARTIAL_REGEX) self.nodes[1].assert_start_raises_init_error(["-addressindex"], "You need to rebuild the database using -reindex to enable -addressindex", match=ErrorMatch.PARTIAL_REGEX)
self.start_node(1, ["-addressindex", "-reindex"]) self.start_node(1, ["-addressindex", "-reindex"])
self.connect_nodes(0, 1) self.connect_nodes(0, 1)
self.sync_all() self.sync_all()

View File

@ -40,14 +40,13 @@ class SpentIndexTest(BitcoinTestFramework):
self.import_deterministic_coinbase_privkeys() self.import_deterministic_coinbase_privkeys()
def run_test(self): def run_test(self):
self.log.info("Test that settings can't be changed without -reindex...") self.log.info("Test that settings can be disabled without -reindex...")
self.stop_node(1) self.restart_node(1, ["-spentindex=0"])
self.nodes[1].assert_start_raises_init_error(["-spentindex=0"], "You need to rebuild the database using -reindex to change -spentindex", match=ErrorMatch.PARTIAL_REGEX)
self.start_node(1, ["-spentindex=0", "-reindex"])
self.connect_nodes(0, 1) self.connect_nodes(0, 1)
self.sync_all() self.sync_all()
self.log.info("Test that settings can't be enabled without -reindex...")
self.stop_node(1) self.stop_node(1)
self.nodes[1].assert_start_raises_init_error(["-spentindex"], "You need to rebuild the database using -reindex to change -spentindex", match=ErrorMatch.PARTIAL_REGEX) self.nodes[1].assert_start_raises_init_error(["-spentindex"], "You need to rebuild the database using -reindex to enable -spentindex", match=ErrorMatch.PARTIAL_REGEX)
self.start_node(1, ["-spentindex", "-reindex"]) self.start_node(1, ["-spentindex", "-reindex"])
self.connect_nodes(0, 1) self.connect_nodes(0, 1)
self.sync_all() self.sync_all()

View File

@ -33,14 +33,13 @@ class TimestampIndexTest(BitcoinTestFramework):
self.sync_all() self.sync_all()
def run_test(self): def run_test(self):
self.log.info("Test that settings can't be changed without -reindex...") self.log.info("Test that settings can be disabled without -reindex...")
self.stop_node(1) self.restart_node(1, ["-timestampindex=0"])
self.nodes[1].assert_start_raises_init_error(["-timestampindex=0"], "You need to rebuild the database using -reindex to change -timestampindex", match=ErrorMatch.PARTIAL_REGEX)
self.start_node(1, ["-timestampindex=0", "-reindex"])
self.connect_nodes(0, 1) self.connect_nodes(0, 1)
self.sync_all() self.sync_all()
self.log.info("Test that settings can't be enabled without -reindex...")
self.stop_node(1) self.stop_node(1)
self.nodes[1].assert_start_raises_init_error(["-timestampindex"], "You need to rebuild the database using -reindex to change -timestampindex", match=ErrorMatch.PARTIAL_REGEX) self.nodes[1].assert_start_raises_init_error(["-timestampindex"], "You need to rebuild the database using -reindex to enable -timestampindex", match=ErrorMatch.PARTIAL_REGEX)
self.start_node(1, ["-timestampindex", "-reindex"]) self.start_node(1, ["-timestampindex", "-reindex"])
self.connect_nodes(0, 1) self.connect_nodes(0, 1)
self.sync_all() self.sync_all()