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.

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

sql: add a regression test for virtual table generation hang #100003

Merged
merged 1 commit into from
Mar 31, 2023

Conversation

yuzefovich
Copy link
Member

This commit adds a regression test for #99753 that verifies that virtual table generation doesn't hang when the worker goroutine returns the query canceled error.

Informs #99753

Release note: None

@cockroach-teamcity
Copy link
Member

This change is Reviewable

This commit adds a regression test for cockroachdb#99753 that verifies that virtual
table generation doesn't hang when the worker goroutine returns the
query canceled error.

Release note: None
@yuzefovich yuzefovich marked this pull request as ready for review March 29, 2023 23:55
@yuzefovich yuzefovich requested review from a team, rafiss and cucaroach March 29, 2023 23:56
@yuzefovich yuzefovich added the backport-23.1.x Flags PRs that need to be backported to 23.1 label Mar 31, 2023
@yuzefovich
Copy link
Member Author

TFTR!

bors r+

@craig
Copy link
Contributor

craig bot commented Mar 31, 2023

Timed out.

@yuzefovich
Copy link
Member Author

Seems like a flake.

bors r+

@craig
Copy link
Contributor

craig bot commented Mar 31, 2023

Build succeeded:

@craig craig bot merged commit 4d77668 into cockroachdb:master Mar 31, 2023
@yuzefovich yuzefovich deleted the vt-regression branch March 31, 2023 05:27
@mgartner
Copy link
Collaborator

blathers backport 23.1.0

@mgartner
Copy link
Collaborator

Ignore the backport to 23.1.0 - didn't realize we were waiting for 23.1.1 for backporting this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-23.1.x Flags PRs that need to be backported to 23.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants