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

Fix Gen4 group/order by with derived table #10074

Merged
merged 4 commits into from
Apr 21, 2022

Conversation

frouioui
Copy link
Member

@frouioui frouioui commented Apr 12, 2022

Description

This PR fixes an issue (observed in #10011) that arises when the planner was rewriting expressions to their derived form, even though they did not need to be rewritten.

Related Issue(s)

Checklist

  • Should this PR be backported?
  • Tests were added or are not required
  • Documentation was added or is not required

@systay systay force-pushed the gen4-group-order-by-derived-table branch from 4334d31 to f71ee6b Compare April 12, 2022 15:35
@derekperkins
Copy link
Member

Does this PR have to land to fix #10011?

@frouioui
Copy link
Member Author

Does this PR have to land to fix #10011?

@derekperkins Yes.

@systay systay force-pushed the gen4-group-order-by-derived-table branch from b1ee25f to b377248 Compare April 20, 2022 07:28
Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
@frouioui frouioui force-pushed the gen4-group-order-by-derived-table branch from b377248 to c616b4c Compare April 20, 2022 13:00
@frouioui frouioui marked this pull request as ready for review April 20, 2022 13:02
@frouioui frouioui marked this pull request as draft April 20, 2022 13:06
Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
@frouioui frouioui marked this pull request as ready for review April 21, 2022 06:37
@frouioui frouioui force-pushed the gen4-group-order-by-derived-table branch from 676694e to 11a584e Compare April 21, 2022 06:37
Signed-off-by: Andres Taylor <andres@planetscale.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vtgate: Gen4 only_full_group_by regression
3 participants