-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
planner: cannot simply outer join if a predicate just refers to the outer table #16444
Conversation
Codecov Report
@@ Coverage Diff @@
## master #16444 +/- ##
===========================================
Coverage 80.4354% 80.4354%
===========================================
Files 506 506
Lines 136701 136701
===========================================
Hits 109956 109956
Misses 18185 18185
Partials 8560 8560 |
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.
LGTM
Please add some labels for this PR. |
Done |
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.
LGTM
/merge |
/run-all-tests |
@fzhedu merge failed. |
/run-integration-copr-test |
/run-integration-{copr}-test |
/run-all-tests |
/merge |
/run-all-tests |
@fzhedu merge failed. |
/merge |
/run-all-tests |
@fzhedu merge failed. |
/merge |
/run-all-tests |
/run-cherry-picker |
Signed-off-by: sre-bot <sre-bot@pingcap.com>
cherry pick to release-2.1 in PR #16491 |
Signed-off-by: sre-bot <sre-bot@pingcap.com>
cherry pick to release-3.0 in PR #16492 |
Signed-off-by: sre-bot <sre-bot@pingcap.com>
cherry pick to release-3.1 in PR #16493 |
cherry pick to release-4.0 in PR #16494 |
What problem does this PR solve?
Issue Number: close #16023
Problem Summary:
Simplying the outer join with predicates shoud not take a predicate which only refers to the outer table
What is changed and how it works?
Proposal: xxx
What's Changed:
How it Works:
Related changes
Check List
Tests
Side effects
Release note