planner: skip plan cache if the plan contains Shuffle operators#42168
planner: skip plan cache if the plan contains Shuffle operators#42168ti-chi-bot merged 3 commits intopingcap:release-6.1from
Conversation
|
[REVIEW NOTIFICATION] This pull request has been approved by:
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. DetailsReviewer can indicate their review by submitting an approval review. |
|
Skipping CI for Draft Pull Request. |
|
/retest |
|
/retest |
planner/core/common_plans.go
Outdated
| childContainTableDual := false | ||
| for _, child := range physicalPlan.Children() { | ||
| childContainTableDual = childContainTableDual || containShuffleOperator(child) | ||
| if childContainTableDual { | ||
| return true | ||
| } | ||
| } | ||
| return false |
There was a problem hiding this comment.
for _, child := range physicalPlan.Children() {
if containShuffuleOperator(child) {
return true
}
}
return false
|
/merge |
|
This pull request has been accepted and is ready to merge. DetailsCommit hash: 7102df9 |
What problem does this PR solve?
Issue Number: close #38335
Problem Summary:
What is changed and how it works?
planner: skip plan cache if the plan contains Shuffle operators
Check List
Tests
Side effects
Documentation
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.