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.

Sign up for GitHub

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

Jump to bottom

sql, bulkio: handle restoring tables into a multi-region database #59804

Closed
arulajmani opened this issue Feb 4, 2021 · 0 comments · Fixed by #65015
Closed

sql, bulkio: handle restoring tables into a multi-region database #59804

arulajmani opened this issue Feb 4, 2021 · 0 comments · Fixed by #65015
Assignees
@pbardea
Labels
A-disaster-recovery A-multiregion Related to multi-region C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-multiregion
Projects
Disaster Recovery Backlog

Comments

@arulajmani
Copy link
Collaborator

arulajmani commented Feb 4, 2021 •
edited by elinorgarcia

The current thinking is to disallow table level backups for multi-region tables (at least for 21.1). Thus the only tables that may be restored into a multi-region database are those that were backed-up from a non-multi-region database. When restoring these to what is now a multi-region database, they should behave as regional by table tables homed in the database's primary region. As such, they should get a locality config on the table descriptor to reflect this.

Followup issue from the discussion with @dt, @pbardea and @ajstorm. cc @awoods187 , you might be interested in this one too.

Epic CRDB-7074
@arulajmani arulajmani added C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) A-multiregion Related to multi-region labels Feb 4, 2021
@ajstorm ajstorm self-assigned this Feb 16, 2021
@Elliebababa Elliebababa mentioned this issue May 12, 2021
Merged
@mwang1026 mwang1026 added this to Triage in Disaster Recovery Backlog via automation May 28, 2021
@mwang1026 mwang1026 moved this from Triage to Backup/Restore/Dump in Disaster Recovery Backlog Jun 1, 2021
@mwang1026 mwang1026 assigned pbardea and unassigned ajstorm Jun 1, 2021
@craig craig bot closed this as completed in 1d0eaa9 Jul 6, 2021
Disaster Recovery Backlog automation moved this from Backup/Restore/Dump to Done Jul 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pbardea pbardea

Labels
A-disaster-recovery A-multiregion Related to multi-region C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-multiregion
Projects
Disaster Recovery Backlog
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

backupccl: allow restoring into multi-region databases Elliebababa/cockroach
5 participants
@jlinder @pbardea @arulajmani @ajstorm @mwang1026