-
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: don't push down null sensitive join conditions (#19620) #21019
planner: don't push down null sensitive join conditions (#19620) #21019
Conversation
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
/run-all-tests |
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
/run-common-test tidb-test=pr/1102 |
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 |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
@ti-srebot merge failed. |
/run-all-tests |
/run-all-tests tidb-test=pr/1102 |
Should merge https://github.com/pingcap/tidb-test/pull/1102 first. |
cherry-pick #19620 to release-4.0
What problem does this PR solve?
Issue Number:
Problem Summary:
What is changed and how it works?
What's Changed:
RowFunc
asInOperand
;InOperand
columns;How it Works:
OtherConditions
of join, and can be correctly evaluated then.Related changes
Check List
Tests
Side effects
N/A
Release note