Skip to content

Book docs for "db vacate"#6747

Open
philrz wants to merge 1 commit intomainfrom
vacate-docs
Open

Book docs for "db vacate"#6747
philrz wants to merge 1 commit intomainfrom
vacate-docs

Conversation

@philrz
Copy link
Contributor

@philrz philrz commented Mar 18, 2026

What's Changing

This updates the db vacate book docs to match what was added in #6706.

Why

The docs being replaced here pre-dated #6706 and were not an accurate representation of what was added.

Details

I'd have liked to include something about when certain vacate attempts will be rejected such as shown in the bottom test in vacate.yaml. I'm told it has something to do with "common ancestors", but I'm struggling with the wording. If someone can propose the right text I'm game. Otherwise happy to leave for later.

While running tests to write this, I spotted #6746.

@philrz philrz requested a review from a team March 18, 2026 21:14
@philrz philrz self-assigned this Mar 18, 2026
Name: "vacate",
Usage: "vacate [options] [timestamp]",
Short: "compact a pool's commit history by removing old commit objects",
Short: "truncate a pool's commit history by removing old commit objects",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Proposing truncate since compact is already a verb attached to other functionality, and also we're deleting not squashing as the command was once envisioned.

@philrz philrz linked an issue Mar 18, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

db vacate & purge

1 participant