Skip to content

Add the archive --purge flag to clean up table files#9095

Merged
macneale4 merged 2 commits intomainfrom
macneale4/archive-purge
Apr 10, 2025
Merged

Add the archive --purge flag to clean up table files#9095
macneale4 merged 2 commits intomainfrom
macneale4/archive-purge

Conversation

@macneale4
Copy link
Copy Markdown
Contributor

@macneale4 macneale4 commented Apr 8, 2025 •
edited
Loading

The dolt archive command is extra safe, and doesn't cleanup table files is converts. The --purge flag will delete table files after the conversion is done.

@macneale4 macneale4 requested a review from reltuk April 8, 2025 22:32
@coffeegoddd
Copy link
Copy Markdown
Contributor

coffeegoddd commented Apr 8, 2025

@macneale4 DOLT

comparing_percentages
100.000000 to 100.000000
version result total
8d2ffa5 ok 5937457
version total_tests
8d2ffa5 5937457
correctness_percentage
100.0

@coffeegoddd
Copy link
Copy Markdown
Contributor

coffeegoddd commented Apr 9, 2025

@macneale4 DOLT

comparing_percentages
100.000000 to 100.000000
version result total
cff2fd3 ok 5937457
version total_tests
cff2fd3 5937457
correctness_percentage
100.0

reltuk
reltuk approved these changes Apr 10, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@reltuk reltuk left a comment

Choose a reason for hiding this comment

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

LGTM!

@macneale4 macneale4 merged commit 728b7c1 into main Apr 10, 2025
21 checks passed
@BrewTestBot BrewTestBot mentioned this pull request Apr 11, 2025
Merged
@tbantle22 tbantle22 deleted the macneale4/archive-purge branch May 28, 2025 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@reltuk reltuk reltuk approved these changes

Assignees

No one assigned

Labels

correctness_approved

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@macneale4 @coffeegoddd @reltuk