-
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
expression: open push down switcher for part of CAST functions #13837
Conversation
Codecov Report
@@ Coverage Diff @@
## master #13837 +/- ##
===========================================
Coverage ? 80.1556%
===========================================
Files ? 474
Lines ? 117066
Branches ? 0
===========================================
Hits ? 93835
Misses ? 15819
Partials ? 7412 |
Signed-off-by: Lonng <heng@lonng.org>
…wn-cast Signed-off-by: Iosmanthus Teng <myosmanthustree@gmail.com>
|
Signed-off-by: Lonng <heng@lonng.org>
Signed-off-by: Iosmanthus Teng <myosmanthustree@gmail.com>
/run-integration-common-test tidb-test=pr/960 |
Signed-off-by: Lonng <heng@lonng.org>
/run-integration-common-test tidb-test=pr/960 |
/run-all-tests tidb-test=pr/960 |
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
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 |
@lonng merge failed. |
/run-integration-common-test tidb-test=pr/960 |
Signed-off-by: Lonng heng@lonng.org
What problem does this PR solve?
Open the CAST push-down switcher and fix some execution plans.
What is changed and how it works?
We have fixed all CAST functions related inconsistent between TiDB and TiKV, it's time to open the switcher default.
Check List
Tests
Release note