Skip to content

tree: fix internal error comparing tuple type with non-tuple type#92635

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
msirek:catchUnsupportedComparison
Nov 30, 2022
Merged

tree: fix internal error comparing tuple type with non-tuple type#92635
craig[bot] merged 1 commit intocockroachdb:masterfrom
msirek:catchUnsupportedComparison

Conversation

@msirek
Copy link
Copy Markdown
Contributor

@msirek msirek commented Nov 29, 2022

Fixes #91532

This fixes comparison of an empty tuple expression, ROW() or (), or
a non-empty tuple expression with a non-tuple type by returning
an error message instead of an internal error.

Release note (bug fix): This fixes an internal error when comparing
a tuple type with a non-tuple type.

@msirek msirek requested review from a team, mgartner and rharding6373 November 29, 2022 00:52
@msirek msirek force-pushed the catchUnsupportedComparison branch from 162b4c5 to 5f2b661 Compare November 29, 2022 06:30
@msirek
Copy link
Copy Markdown
Contributor Author

msirek commented Nov 29, 2022

Maybe hold off on review... a few changes coming.

@msirek msirek force-pushed the catchUnsupportedComparison branch from 5f2b661 to 9ad8a6f Compare November 29, 2022 18:09
Copy link
Copy Markdown
Contributor

@mgartner mgartner left a comment

Choose a reason for hiding this comment

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

LGTM

@rytaft
Copy link
Copy Markdown
Collaborator

rytaft commented Nov 29, 2022

Consider adding a backport label for 22.2 and 22.1

Fixes cockroachdb#91532

This fixes comparison of an empty tuple expression, ROW() or (), or
a non-empty tuple expression with a non-tuple type by returning
an error message instead of an internal error.

Release note (bug fix): This fixes an internal error when comparing
a tuple type with a non-tuple type.
@msirek msirek force-pushed the catchUnsupportedComparison branch from 9ad8a6f to 7b473a8 Compare November 29, 2022 22:18
Copy link
Copy Markdown
Collaborator

@rytaft rytaft left a comment

Choose a reason for hiding this comment

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

LGTM

@msirek
Copy link
Copy Markdown
Contributor Author

msirek commented Nov 29, 2022

TFTRs!
bors r=rytaft,mgartner

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Nov 30, 2022

Build succeeded:

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

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: query causes Internal Error

4 participants