-
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: handle single partition in IndexJoin correctly #12581
Conversation
Codecov Report
@@ Coverage Diff @@
## master #12581 +/- ##
================================================
- Coverage 79.9006% 79.8622% -0.0384%
================================================
Files 462 462
Lines 104864 104833 -31
================================================
- Hits 83787 83722 -65
- Misses 14886 14914 +28
- Partials 6191 6197 +6 |
/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
@eurekaka please resolve the conflicts. |
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-all-tests |
cherry pick to release-3.1 failed |
cherry pick to release-2.1 failed |
cherry pick to release-3.0 failed |
Conflicts: planner/core/exhaust_physical_plans.go
Conflicts: planner/core/exhaust_physical_plans.go
Conflicts: executor/index_lookup_join_test.go planner/core/exhaust_physical_plans.go
What problem does this PR solve?
Fix #12549
What is changed and how it works?
Set
isPartition
andphysicalTableID
forPhysicalIndexScan
correctly when the inner child is a single partition of a partitioned table.Check List
Tests
Code changes
N/A
Side effects
N/A
Related changes
Release note
N/A