Skip to content

sql: deflake unique logic test#78984

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
mgartner:deflake-unique
Mar 29, 2022
Merged

sql: deflake unique logic test#78984
craig[bot] merged 1 commit intocockroachdb:masterfrom
mgartner:deflake-unique

Conversation

@mgartner
Copy link
Copy Markdown
Contributor

PR #78685 changes the query plan of one query in a logic test in a way
that makes the test flakey. This commit guarantees that the test cannot
be flakey, regardless of the query plan.

Release note: None

PR cockroachdb#78685 changes the query plan of one query in a logic test in a way
that makes the test flakey. This commit guarantees that the test cannot
be flakey, regardless of the query plan.

Release note: None
@mgartner mgartner requested a review from a team March 29, 2022 20:40
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Member

@yuzefovich yuzefovich left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @mgartner)

@mgartner
Copy link
Copy Markdown
Contributor Author

bors r+

p=999

@mgartner
Copy link
Copy Markdown
Contributor Author

bors r+ p=999

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 29, 2022

Already running a review

@mgartner
Copy link
Copy Markdown
Contributor Author

bors cancel

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 29, 2022

Canceled.

@mgartner
Copy link
Copy Markdown
Contributor Author

bors r+ p=999

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 29, 2022

Build succeeded:

@craig craig bot merged commit b154814 into cockroachdb:master Mar 29, 2022
@mgartner mgartner deleted the deflake-unique branch March 29, 2022 22:44
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.

3 participants