bitshares / bitshares-core

BitShares Blockchain node and command-line wallet
https://bitshares.github.io/
Other
1.17k stars 648 forks source link

Variable recently_missed_count is no longer in use #2500

Open abitmore opened 3 years ago

abitmore commented 3 years ago

User Story

The variable recently_missed_count in dynamic global properties is no longer in use since https://github.com/bitshares/bitshares-core/commit/3ded0d4e773594361a7fe2db40cef0518c257f78.

It was useful for playing with checkpoints, see https://github.com/bitshares/bitshares-core/commit/73cc012db90fce64671a6669af3cd7671a49be28. Now the trick to restart block production with a dummy checkpoint (a known block in the past with its correct ID) no longer works when undo_db and fork_db are growing. Another trick still works -- setting an unknown block in the future as a checkpoint -- but it is too dangerous in production since validation will be skipped.

Perhaps implement another trick, or remove related code.

We did not use checkpoints in the BitShares Mainnet so far.

Impacts Describe which portion(s) of BitShares Core may be impacted by your request. Please tick at least one box.

CORE TEAM TASK LIST