tidb
tidb copied to clipboard
planner: stop pushing Agg down through Projection if substitution fail (#50932)
This is an automated cherry-pick of #50932
What problem does this PR solve?
Issue Number: close #50926
Problem Summary: planner: stop pushing Agg down through Projection if substitution fail
What changed and how does it work?
planner: stop pushing Agg down through Projection if substitution fail
Check List
Tests
- [x] Unit test
- [ ] Integration test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test
- [ ] I checked and no code files have been changed.
Side effects
- [ ] Performance regression: Consumes more CPU
- [ ] Performance regression: Consumes more Memory
- [ ] Breaking backward compatibility
Documentation
- [ ] Affects user behaviors
- [ ] Contains syntax changes
- [ ] Contains variable changes
- [ ] Contains experimental features
- [ ] Changes MySQL compatibility
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.
None
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fixdb, qw4990
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [qw4990]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
[LGTM Timeline notifier]
Timeline:
/test mysql-test
@qw4990: No presubmit jobs available for pingcap/[email protected]
In response to this:
/test mysql-test
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.
/test mysql-test
@qw4990: No presubmit jobs available for pingcap/[email protected]
In response to this:
/test mysql-test
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.
Codecov Report
:exclamation: No coverage uploaded for pull request base (
release-7.1@6d13674). Click here to learn what that means. The diff coverage is47.0588%.
Additional details and impacted files
@@ Coverage Diff @@
## release-7.1 #51135 +/- ##
================================================
Coverage ? 73.3987%
================================================
Files ? 1209
Lines ? 378896
Branches ? 0
================================================
Hits ? 278105
Misses ? 83014
Partials ? 17777