Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FreshEyes] Prune locks #12

Open
wants to merge 10 commits into
base: bitcoin-fresheyes-staging-master-19463
Choose a base branch
from

Conversation

adamjonas
Copy link
Owner

The author luke-jr wrote the following PR called Prune locks, issue number 19463 in bitcoin/bitcoin cloned by FreshEyes below:

This adds the ability to have any number of internal or external locks to prevent pruning specific block ranges.

Some examples where this would be useful:

  • Ensuring any known Core wallet won't become unable to sync due to pruning when it is unloaded. (Included in this PR)
  • Ensuring pruning doesn't go too far for a desired-functional wallet backup. (Questionable, since you probably lose the node if you need the backup anyway?)
  • Allowing users to temporarily disable block indexes and filters, without pruning cutting them off from catching up later.
  • Avoiding pruning out from under external wallets (Armory, EPS, etc) that might need block data.
  • External block indexes/filters, etc.

By design, users retain complete control over prune locks, and can delete them out from under applications using them. This is to avoid circumstances where a prune lock has been set, by an application that may no longer be used.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-655272801 at 2020/07/08, 04:15:29 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-655895193 at 2020/07/09, 04:41:02 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-655896520 at 2020/07/09, 04:45:27 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-656214469 at 2020/07/09, 16:04:56 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-664824305 at 2020/07/28, 07:18:07 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-664842207 at 2020/07/28, 07:58:12 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-665806354 at 2020/07/29, 17:45:35 UTC.

Copy link

This is an OUTDATED review comment as the original pull request may have been rebased or force-pushed

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#discussion_r462515631 at 2020/07/29, 18:52:35 UTC.

Copy link

@fresheyes-staging-bot fresheyes-staging-bot bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An author reviewed and commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#pullrequestreview-457798826 at 2020/07/29, 18:52:35 UTC.

Copy link

This is an OUTDATED review comment as the original pull request may have been rebased or force-pushed

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#discussion_r462544433 at 2020/07/29, 19:44:15 UTC.

Copy link

@fresheyes-staging-bot fresheyes-staging-bot bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An author reviewed and commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#pullrequestreview-457833938 at 2020/07/29, 19:44:15 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-822089807 at 2021/04/19, 00:01:15 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-875178165 at 2021/07/07, 00:39:56 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-883492085 at 2021/07/20, 15:36:59 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-1030288247 at 2022/02/04, 19:31:32 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-1312366526 at 2022/11/12, 05:04:35 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-1312406524 at 2022/11/12, 08:11:33 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-1312527049 at 2022/11/12, 16:58:18 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-1522002830 at 2023/04/25, 15:29:41 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-1626446019 at 2023/07/08, 01:58:04 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-1728053831 at 2023/09/20, 16:15:13 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-1728056173 at 2023/09/20, 16:16:51 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-1867040193 at 2023/12/21, 23:13:21 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-2061864564 at 2024/04/17, 17:43:16 UTC.

Copy link

An author commented here with:

  • comment link https://github.com/bitcoin/bitcoin/pull/19463#issuecomment-2071523021 at 2024/04/23, 06:44:43 UTC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants