Skip to content

backupccl: adjust a test to run for secondary tenant codec too#104820

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
yuzefovich:mt-backup-test
Jun 14, 2023
Merged

backupccl: adjust a test to run for secondary tenant codec too#104820
craig[bot] merged 1 commit intocockroachdb:masterfrom
yuzefovich:mt-backup-test

Conversation

@yuzefovich
Copy link
Copy Markdown
Member

Fixes: #82882.

Release note: None

@yuzefovich yuzefovich requested review from knz and stevendanna June 13, 2023 22:08
@yuzefovich yuzefovich requested a review from a team as a code owner June 13, 2023 22:08
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
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.

Thanks!

@yuzefovich
Copy link
Copy Markdown
Member Author

TFTRs!

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jun 14, 2023

Build failed (retrying...):

craig bot pushed a commit that referenced this pull request Jun 14, 2023
104820: backupccl: adjust a test to run for secondary tenant codec too r=yuzefovich a=yuzefovich

Fixes: #82882.

Release note: None

104823: row: remove leftover reference to TestingSQLCodec in a test r=yuzefovich a=yuzefovich

This was missed in #82890.

Addresses: #48123.
Epic: None

Release note: None

104828: base: remove OptionalNodeIDErr r=yuzefovich a=yuzefovich

This method was only used in one place where we need to get the SQL instance ID of the gateway. That place has been refactored to pass that ID explicitly from the DistSQLPlanner.

Addresses: #100826.
Epic: None

Release note: None

Co-authored-by: Yahor Yuzefovich <yahor@cockroachlabs.com>
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jun 14, 2023

Build failed (retrying...):

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jun 14, 2023

Build succeeded:

@craig craig bot merged commit 1d3c777 into cockroachdb:master Jun 14, 2023
@yuzefovich yuzefovich deleted the mt-backup-test branch June 14, 2023 19:51
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.

backupccl: TestPublicIndexTableSpans should exercise secondary tenant keyspans

4 participants