Skip to content

Commit

Permalink
Merge #75207
Browse files Browse the repository at this point in the history
75207: backupccl: call AllocateIDs after adding index mutations r=adityamaru a=stevendanna

While we expect that all of the index mutations added here to have
fully populated IDs and columns. Changes to the AddIndexMutation API
will soon require that AllocateIDs is called after any add index
mutation.

Release note: None

Co-authored-by: Steven Danna <[email protected]>
  • Loading branch information
craig[bot] and stevendanna committed Jan 21, 2022
2 parents 72655fd + dc2d4ea commit 61fa0d5
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions pkg/ccl/backupccl/restore_job.go
Original file line number Diff line number Diff line change
Expand Up @@ -1853,6 +1853,9 @@ func (r *restoreResumer) publishDescriptors(
return err
}
}
if err := mutTable.AllocateIDs(ctx); err != nil {
return err
}
allMutDescs = append(allMutDescs, mutTable)
newTables = append(newTables, mutTable.TableDesc())
// For cluster restores, all the jobs are restored directly from the jobs
Expand Down

0 comments on commit 61fa0d5

Please sign in to comment.