Correct '-dbcache' to '-prune' #29718

pull BenWestgate wants to merge 1 commits into bitcoin:master from BenWestgate:patch-2 changing 1 files +1 −1
  1. BenWestgate commented at 6:42 AM on March 25, 2024: contributor

    This looks like a typo, '-prune' is doubled not the '-dbcache' by my understanding.

  2. correct '-dbcache' to '-prune' 416b9d9427
  3. DrahtBot commented at 6:42 AM on March 25, 2024: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage

    For detailed information about the code coverage, see the test coverage report.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK Sjors, fjahr

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #29553 (assumeutxo: Add dumptxoutset height param, remove shell scripts by fjahr)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  4. Sjors commented at 11:21 AM on March 25, 2024: member

    ACK 416b9d9427c017fe7eb2975ca76f6a89ba24ab58

    The minimum -dbcache value is 4 MiB, whereas the minimum -prune is 550 MiB.

    In BlockManager::FindFilesToPrune there's this:

        const auto target = std::max(
            MIN_DISK_SPACE_FOR_BLOCK_FILES, GetPruneTarget() / chainman.GetAll().size());
    

    During snapshot handling chainman.GetAll().size() is 2.

    prune_target is set by -prune

    MIN_DISK_SPACE_FOR_BLOCK_FILES is 550 MiB

    This is line allocates half the -prune value to the background sync and half of it to the foreground sync, but never less than 550 MiB to each.

    cc @jamesob / @fjahr

  5. fjahr commented at 1:15 PM on March 25, 2024: contributor

    ACK 416b9d9427c017fe7eb2975ca76f6a89ba24ab58

  6. fanquake merged this on Mar 25, 2024
  7. fanquake closed this on Mar 25, 2024

  8. BenWestgate deleted the branch on Mar 25, 2024
  9. bitcoin locked this on Mar 25, 2025

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-29 03:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me