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
opt: fix internal error when aggregating on a virtual table #44692
Conversation
Recently introduced logic allows projecting any columns if we are grouping by the PK of the table. This code generates an internal error for virtual tables, which have no primary index. Note that in this case we expect an error. Fixes cockroachdb#44659. Release note (bug fix): fixed "no indexes" internal error in some cases when we GROUP BY on a virtual table.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @rytaft)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 2 of 2 files at r1.
Reviewable status: complete! 2 of 0 LGTMs obtained
bors r+ |
Build failed (retrying...) |
44692: opt: fix internal error when aggregating on a virtual table r=RaduBerinde a=RaduBerinde Recently introduced logic allows projecting any columns if we are grouping by the PK of the table. This code generates an internal error for virtual tables, which have no primary index. Note that in this case we expect an error. Fixes #44659. Release note (bug fix): fixed "no indexes" internal error in some cases when we GROUP BY on a virtual table. Co-authored-by: Radu Berinde <radu@cockroachlabs.com>
Build succeeded |
Recently introduced logic allows projecting any columns if we are
grouping by the PK of the table. This code generates an internal error
for virtual tables, which have no primary index. Note that in this
case we expect an error.
Fixes #44659.
Release note (bug fix): fixed "no indexes" internal error in some
cases when we GROUP BY on a virtual table.