Page MenuHomePhabricator

walletdb: remove fAggressive from Salvage
ClosedPublic

Authored by Fabien on Dec 3 2020, 16:09.

Details

Reviewers
majcosta
Group Reviewers
Restricted Project
Commits
rABC186cb7d12277: walletdb: remove fAggressive from Salvage
Summary
The only call to Salvage set fAggressive = true so remove that parameter
and always use DB_AGGRESSIVE

Partial backport (5/11) of core PR18918:
https://github.com/bitcoin/bitcoin/pull/18918/commits/07250b8dcebe2b97ed0fd900ad35cba4091b8ecf

Depends on D8591.

Test Plan
ninja all check-all

Diff Detail

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

Event Timeline

Fabien requested review of this revision.Dec 3 2020, 16:09

Tail of the build log:

[369/425] Running utility command for check-bitcoin-settings_tests
[370/425] bitcoin: testing timedata_tests
[371/425] Running utility command for check-bitcoin-timedata_tests
[372/425] bitcoin: testing streams_tests
[373/425] bitcoin: testing undo_tests
[374/425] Running utility command for check-bitcoin-streams_tests
[375/425] bitcoin: testing util_threadnames_tests
[376/425] Running utility command for check-bitcoin-undo_tests
[377/425] Running utility command for check-bitcoin-util_threadnames_tests
[378/425] bitcoin: testing validation_chainstatemanager_tests
[379/425] Running utility command for check-bitcoin-validation_chainstatemanager_tests
[380/425] bitcoin: testing compilerbug_tests
[381/425] bitcoin: testing txvalidationcache_tests
[382/425] bitcoin: testing serialize_tests
[383/425] Running utility command for check-bitcoin-txvalidationcache_tests
[384/425] Running utility command for check-bitcoin-compilerbug_tests
[385/425] Running utility command for check-bitcoin-serialize_tests
[386/425] bitcoin: testing radix_tests
[387/425] bitcoin: testing cashaddr_tests
[388/425] Running utility command for check-bitcoin-radix_tests
[389/425] Running utility command for check-bitcoin-cashaddr_tests
[390/425] bitcoin: testing validationinterface_tests
[391/425] bitcoin: testing checkpoints_tests
[392/425] Running utility command for check-bitcoin-validationinterface_tests
[393/425] bitcoin: testing crypto_tests
[394/425] Running utility command for check-bitcoin-checkpoints_tests
[395/425] bitcoin: testing blockcheck_tests
[396/425] Running utility command for check-bitcoin-blockcheck_tests
[397/425] Running utility command for check-bitcoin-crypto_tests
[398/425] Running bitcoin-qt test suite
PASSED: bitcoin-qt test suite
[399/425] bitcoin: testing validation_tests
[400/425] Running utility command for check-bitcoin-validation_tests
[401/425] bitcoin: testing getarg_tests
[402/425] bitcoin: testing ref_tests
[403/425] Running utility command for check-bitcoin-getarg_tests
[404/425] Running utility command for check-bitcoin-ref_tests
[405/425] bitcoin: testing script_tests
[406/425] Running utility command for check-bitcoin-script_tests
[407/425] bitcoin: testing monolith_opcodes_tests
[408/425] Running utility command for check-bitcoin-monolith_opcodes_tests
[409/425] bitcoin: testing skiplist_tests
[410/425] Running utility command for check-bitcoin-skiplist_tests
[411/425] bitcoin: testing coinselector_tests
[412/425] Running utility command for check-bitcoin-coinselector_tests
[413/425] bitcoin: testing util_tests
[414/425] Running utility command for check-bitcoin-util_tests
[415/425] bitcoin: testing op_reversebytes_tests
[416/425] Running utility command for check-bitcoin-op_reversebytes_tests
[417/425] bitcoin: testing transaction_tests
[418/425] Running utility command for check-bitcoin-transaction_tests
[419/425] bitcoin: testing coins_tests
[420/425] Running utility command for check-bitcoin-coins_tests
[421/425] Running bitcoin test suite
PASSED: bitcoin test suite
[422/425] secp256k1: testing secp256k1-tests
[423/425] Running secp256k1 test suite
PASSED: secp256k1 test suite
ninja: build stopped: cannot make progress due to previous errors.
Build build-clang failed with exit code 1
This revision is now accepted and ready to land.Dec 3 2020, 18:47
This revision was landed with ongoing or failed builds.Dec 7 2020, 08:34
This revision was automatically updated to reflect the committed changes.