Skip to content

Conversation

msbutler
Copy link
Collaborator

@msbutler msbutler commented Oct 17, 2025

This helper will be used by full, inc and compacted backups. This helper will
make it much easier to support locality aware compacted backups.

Epic: none

Release note: none

@msbutler msbutler self-assigned this Oct 17, 2025
Copy link

blathers-crl bot commented Oct 17, 2025

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@msbutler msbutler changed the title backup: move partition prefix to backupbase package backup: introduce unified WriteBackupMetadata helper Oct 17, 2025
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@msbutler msbutler marked this pull request as ready for review October 17, 2025 20:18
@msbutler msbutler requested a review from a team as a code owner October 17, 2025 20:18
@msbutler msbutler requested review from jeffswenson and kev-cao and removed request for a team and jeffswenson October 17, 2025 20:18
This helper will be used by full, inc and compacted backups. This helper will
make it much easier to support locality aware compacted backups.

Epic: none

Release note: none
Epic: none

Release note: none
@msbutler msbutler force-pushed the butler-unify-backup-metadata-write-2 branch from 18f12a1 to c87fa68 Compare October 17, 2025 21:03
Copy link
Contributor

@kev-cao kev-cao left a comment

Choose a reason for hiding this comment

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

LGTM!

@msbutler
Copy link
Collaborator Author

TFTR!

bors r=kev-cao

@craig
Copy link
Contributor

craig bot commented Oct 20, 2025

@craig craig bot merged commit 4657cdd into cockroachdb:master Oct 20, 2025
24 checks passed
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.

3 participants