Skip to content

sql/tests: TestRandomSyntaxSchemaChangeColumn fails due to timeout#64877

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
fqazi:randomColumnChangeFix
May 25, 2021
Merged

sql/tests: TestRandomSyntaxSchemaChangeColumn fails due to timeout#64877
craig[bot] merged 1 commit intocockroachdb:masterfrom
fqazi:randomColumnChangeFix

Conversation

@fqazi
Copy link
Copy Markdown
Collaborator

@fqazi fqazi commented May 7, 2021

Fixes: #63393

Previously, there was a 1 minute timeout (in team city) on the DDL
alter statements run during this test under team city. With that
time limit we would observe failures on certain ALTER COLUMN ADD
statements, because the statement timeout could be too short.
To address this, this patch bumps the random column changes
statement timeout to 2 minutes, which resolves these failures.

Release note: None

Fixes: cockroachdb#63393

Previously, there was a 1 minute timeout (in team city) on the DDL
alter statements run during this test under team city. With that
time limit we would observe failures on certain ALTER COLUMN ADD
statements, because the statement timeout could be too short.
To address this, this patch bumps the random column changes
statement timeout to 2 minutes, which resolves these failures.

Release note: None
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@fqazi fqazi requested a review from a team May 18, 2021 14:16
Copy link
Copy Markdown

@postamar postamar left a comment

Choose a reason for hiding this comment

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

Seems reasonable!

@fqazi
Copy link
Copy Markdown
Collaborator Author

fqazi commented May 25, 2021

bors r=postamar

@craig
Copy link
Copy Markdown
Contributor

craig bot commented May 25, 2021

Build succeeded:

@craig craig bot merged commit 245c316 into cockroachdb:master May 25, 2021
@rafiss
Copy link
Copy Markdown
Collaborator

rafiss commented May 26, 2021

Thanks for this fix! It looks we'll want to backport it, since the test is also failing on other branches. #65632 #63944 (and failing on 20.1 also #63202 but there are no more scheduled 20.1 releases, so we likely don't need to backport to 20.1)

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.

sql/tests: TestRandomSyntaxSchemaChangeColumn failed

4 participants