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

backupccl: add random data to tables in TestBackupRestoreDataRoundtrips #77602

Merged
merged 1 commit into from
Jul 14, 2022

Conversation

msbutler
Copy link
Collaborator

@msbutler msbutler commented Mar 10, 2022

Previously, TestBackupRestoreDataRoundtrips did not test if data in the
randomly created tables were preserved after a backup restore roundtrip.
This patch tries to add random data to each table and tests that the data is
preserved across the roundtrip. In addition, this pr moves this test
to its own package because the test takes a long time to run.

Fixes #72345

Release justification: changes to test
Release note: None

@msbutler msbutler requested review from dt, jordanlewis, adityamaru and a team March 10, 2022 15:05
@msbutler msbutler self-assigned this Mar 10, 2022
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Member

@jordanlewis jordanlewis left a comment

Choose a reason for hiding this comment

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

Cool! Looks good, but I'll leave the review to your team :)

@msbutler msbutler force-pushed the butler-rand-backup-restore branch from 2f3716a to 83d7af0 Compare July 11, 2022 11:26
@msbutler msbutler requested a review from a team July 11, 2022 11:26
@msbutler msbutler force-pushed the butler-rand-backup-restore branch from 83d7af0 to db08353 Compare July 11, 2022 12:39
Copy link
Collaborator

@stevendanna stevendanna left a comment

Choose a reason for hiding this comment

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

Apologies this sat for so long. LGTM.

pkg/ccl/backupccl/backuprand/main_test.go Outdated Show resolved Hide resolved
pkg/ccl/backupccl/backuputils/testutils.go Outdated Show resolved Hide resolved
@msbutler msbutler force-pushed the butler-rand-backup-restore branch 4 times, most recently from 3849ccb to 7682ac0 Compare July 14, 2022 12:26
Previously, TestBackupRestoreDataRoundtrips did not test if data in the
randomly created tables were preserved after a backup restore roundtrip.
This patch tries to add random data to each table and tests that the data is
preserved across the roundtrip. In addition, this pr moves this test
to its own package because the test takes a long time to run.

Release note: None
@msbutler msbutler force-pushed the butler-rand-backup-restore branch from 7682ac0 to a4a70f3 Compare July 14, 2022 17:38
@msbutler
Copy link
Collaborator Author

bors r=stevendanna

@craig
Copy link
Contributor

craig bot commented Jul 14, 2022

Build succeeded:

@craig craig bot merged commit 2546b2a into cockroachdb:master Jul 14, 2022
@msbutler msbutler deleted the butler-rand-backup-restore branch July 18, 2022 13:18
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.

bulkio: follow through on random testing
4 participants