Page MenuHomePhabricator

implements different disk sizes for different networks on intro
ClosedPublic

Authored by deadalnix on Apr 16 2020, 22:21.

Details

Summary
  • Creates m_assumed_blockchain_size and m_assumed_chain_state_size on CChainParams.
  • Implements access to CChainParams' m_assumed_blockchain_size and m_assumed_chain_state_size on node interface.
  • Implements m_assumed_blockchain_size and m_assumed_chain_state_size on qt/intro via node interface.
  • Updates release process document with the new CChainParam's values.

Backport of core PR13216.

Test Plan
ninja check-all

Build the win64 client and run it on my windows machine.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Fabien requested changes to this revision.Apr 17 2020, 08:18
Fabien added a subscriber: Fabien.
Fabien added inline comments.
src/chainparams.cpp
196 ↗(On Diff #18876)

It was 220 and 4. This is a bit overestimated (I have 156G/2.3G), but would be weird to have the size reduced over time.

420 ↗(On Diff #18876)

Current testnet is about 43GB/1.2GB

This revision now requires changes to proceed.Apr 17 2020, 08:18
deadalnix added inline comments.
src/chainparams.cpp
196 ↗(On Diff #18876)

It really doesn't matter at this point and getting these number really is a task for a bot, not a human.

This revision is now accepted and ready to land.Apr 17 2020, 14:13