-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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: fix order by sub-query couldn't find outer correlated columns[DNM] #33639
Conversation
Signed-off-by: arenatlx <arenatlx@arenatlxdeMBP.lan>
Signed-off-by: arenatlx <arenatlx@arenatlxdeMacBook-Pro.local>
Signed-off-by: arenatlx <arenatlx@arenatlxdeMacBook-Pro.local>
Signed-off-by: arenatlx <arenatlx@arenatlxdeMacBook-Pro.local>
[REVIEW NOTIFICATION] This pull request has not been approved. To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
arenatlx seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
Code Coverage Details: https://codecov.io/github/pingcap/tidb/commit/cef6ed94d04e7c1099c74e950c40369b4c8f0110 |
What problem does this PR solve?
Issue Number: close #26945
Problem Summary:
What is changed and how it works?
just used to test CLA
fetch the correlated column in order-by subquery out to select fields, building it as an Auxiliary one, which will be stored as p.schema() (passing as outer schema when enter subquery) when we try to build order-by clause. Otherwise it couldn't find that correlated column.
Check List
Tests
Side effects
Documentation
Release note