Skip to content

options: add Unsafe.AllowMissingWALDir option#5478

Merged
RaduBerinde merged 1 commit intocockroachdb:masterfrom
RaduBerinde:allow-missing-wal-dirs
Oct 23, 2025
Merged

options: add Unsafe.AllowMissingWALDir option#5478
RaduBerinde merged 1 commit intocockroachdb:masterfrom
RaduBerinde:allow-missing-wal-dirs

Conversation

@RaduBerinde
Copy link
Copy Markdown
Member

This unsafe option allows opening a store after moving or losing the
previous WAL dir. This is understood to be risky and have associated
data loss if any of the necessary WALs are not in the new directory.

Fixes #5421

@RaduBerinde RaduBerinde requested a review from jbowens October 21, 2025 21:37
@RaduBerinde RaduBerinde requested a review from a team as a code owner October 21, 2025 21:37
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@RaduBerinde
Copy link
Copy Markdown
Member Author

CC @annikiforov - this should address your problem, correct?

This unsafe option allows opening a store after moving or losing the
previous WAL dir. This is understood to be risky and have associated
data loss if any of the necessary WALs are not in the new directory.

Fixes cockroachdb#5421
@RaduBerinde RaduBerinde force-pushed the allow-missing-wal-dirs branch from 4a14f92 to 6cf1e8a Compare October 22, 2025 01:32
@annikiforov
Copy link
Copy Markdown

CC @annikiforov - this should address your problem, correct?

Absolutely, perfect!

Copy link
Copy Markdown
Contributor

@xinhaoz xinhaoz left a comment

Choose a reason for hiding this comment

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

@xinhaoz reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @jbowens)

@RaduBerinde
Copy link
Copy Markdown
Member Author

TFTR!

@RaduBerinde RaduBerinde merged commit 67e0ff0 into cockroachdb:master Oct 23, 2025
8 checks passed
@RaduBerinde RaduBerinde deleted the allow-missing-wal-dirs branch October 23, 2025 22:02
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.

db: Allow to rename WALDir

4 participants