-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Do not multiply AggregateRandom
in JOIN
s
#11633
Conversation
Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
Review ChecklistHello reviewers! 👋 Please follow this checklist when reviewing this Pull Request. General
If a new flag is being introduced:
If a workflow is added or modified:
Bug fixes
Non-trivial changes
New/Existing features
Backward compatibility
|
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.
❤️
@frouioui : A quick question about BTW: 你會說中文? :-P |
@tokikanno, the
我会一点点 不过我的中文不好 我的大学在北京 |
@tokikanno, regarding doing a patch release: we will wait until at least the 3rd week of November to do a patch release. Since we recently released v15.0.0 we want to wait until we get more bug fixes. |
Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
@systay, you added the |
Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
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.
👍 on the release notes changes.
I was unable to backport this Pull Request to the following branches: |
* Do not multiply AggregateRandom in JOIN Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> * Update the releases notes with new known issue Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
* Do not multiply AggregateRandom in JOIN Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> * Update the releases notes with new known issue Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
* Do not multiply `AggregateRandom` in `JOIN`s (#11633) * Do not multiply AggregateRandom in JOIN Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> * Update the releases notes with new known issue Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> * Addition of the release notes for v14.0.4 Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> * set sql mode in TestAggregateRandom Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> * update plan test to remove table used json object Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
* Do not multiply `AggregateRandom` in `JOIN`s (#11633) * Do not multiply AggregateRandom in JOIN Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> * Update the releases notes with new known issue Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> * Addition of the release notes for v15.0.1 Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> * Fix format issue in the release notes summary Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> * set sql mode in TestAggregateRandom Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
…io#1308) * Do not multiply `AggregateRandom` in `JOIN`s (vitessio#11633) * Do not multiply AggregateRandom in JOIN Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> * Update the releases notes with new known issue Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> Signed-off-by: Andres Taylor <andres@planetscale.com> * clean up all tables after running e2e tests Signed-off-by: Andres Taylor <andres@planetscale.com> * Remove TestOrderByCount test Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr> Signed-off-by: Andres Taylor <andres@planetscale.com> Co-authored-by: FlorentP <35779988+frouioui@users.noreply.github.com> Co-authored-by: Florent Poinsard <florent.poinsard@outlook.fr>
Description
This Pull Request fixes the issue described in #11625. When pushing aggregation on a join, we were skipping the multiplication of results for
min
andmax
aggregations, however, we also need to skip the multiplication forrandom
aggregations. Random aggregations simply need to get the column form one of the join's sides without multiplying it to the other side of the join.Related Issue(s)
Checklist