Skip to content

sql: enable tenant testing for schema_changer tests#141213

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
shubhamdhama:sql-tenant-testing-schema-changer
Feb 14, 2025
Merged

sql: enable tenant testing for schema_changer tests#141213
craig[bot] merged 1 commit intocockroachdb:masterfrom
shubhamdhama:sql-tenant-testing-schema-changer

Conversation

@shubhamdhama
Copy link
Copy Markdown
Contributor

Previously all tests in this file were disabled for multitenancy.

(Continuation of #140447 for schema_changer_test.go)

Informs: #140446
Epic: CRDB-38970
Release note: None

Previously all tests in this file were disabled for multitenancy.

(Continuation of cockroachdb#140447 for `schema_changer_test.go`)

Informs: cockroachdb#140446
Epic: CRDB-38970
Release note: None
@shubhamdhama shubhamdhama requested a review from a team as a code owner February 12, 2025 05:59
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@shubhamdhama
Copy link
Copy Markdown
Contributor Author

The affected tests in this PR don't take that much longer in shared tenant mode compared to single tenant (70s vs ~80s). But, like a lot of other tests, external process mode is a different story (~180s).

Copy link
Copy Markdown
Contributor

@cthumuluru-crdb cthumuluru-crdb left a comment

Choose a reason for hiding this comment

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

LGTM

@shubhamdhama
Copy link
Copy Markdown
Contributor Author

TFTR!

bors r=cthumuluru-crdb,rafiss

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Feb 14, 2025

@craig craig bot merged commit e08af3f into cockroachdb:master Feb 14, 2025
19 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.

4 participants