Skip to content
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

Merged
merged 10 commits into from
Jan 2, 2020
Merged

expression: open push down switcher for part of CAST functions #13837

merged 10 commits into from
Jan 2, 2020

Conversation

lonng
Copy link
Contributor

@lonng lonng commented Dec 1, 2019

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

  • Unit test
  • Integration test

Release note

  • No need

@lonng lonng requested a review from a team as a code owner December 1, 2019 04:32
@ghost ghost requested review from qw4990 and XuHuaiyu and removed request for a team December 1, 2019 04:32
@codecov
Copy link

codecov bot commented Dec 1, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@b1c08ee). Click here to learn what that means.
The diff coverage is 66.6666%.

@@             Coverage Diff             @@
##             master     #13837   +/-   ##
===========================================
  Coverage          ?   80.1556%           
===========================================
  Files             ?        474           
  Lines             ?     117066           
  Branches          ?          0           
===========================================
  Hits              ?      93835           
  Misses            ?      15819           
  Partials          ?       7412

lonng and others added 2 commits December 1, 2019 13:38
Signed-off-by: Lonng <heng@lonng.org>
…wn-cast

Signed-off-by: Iosmanthus Teng <myosmanthustree@gmail.com>
@lonng lonng requested a review from a team as a code owner December 31, 2019 05:42
@ghost ghost requested review from eurekaka and winoros and removed request for a team December 31, 2019 05:42
@iosmanthus iosmanthus changed the title [DNM] expression: open CAST function push down switcher expression: open CAST function push down switcher Dec 31, 2019
@iosmanthus
Copy link
Contributor

copr-test may fail, more detail on: #14280

lonng and others added 6 commits December 31, 2019 14:04
Signed-off-by: Lonng <heng@lonng.org>
Signed-off-by: Lonng <heng@lonng.org>
Signed-off-by: Lonng <heng@lonng.org>
Signed-off-by: Iosmanthus Teng <myosmanthustree@gmail.com>
@iosmanthus
Copy link
Contributor

/run-integration-common-test tidb-test=pr/960

Signed-off-by: Lonng <heng@lonng.org>
@lonng
Copy link
Contributor Author

lonng commented Dec 31, 2019

/run-integration-common-test tidb-test=pr/960

@lonng
Copy link
Contributor Author

lonng commented Dec 31, 2019

/run-all-tests tidb-test=pr/960

@XuHuaiyu XuHuaiyu changed the title expression: open CAST function push down switcher expression: open push down switcher for part of CAST functions Jan 2, 2020
Copy link
Contributor

@XuHuaiyu XuHuaiyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@qw4990 qw4990 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@qw4990 qw4990 added the status/can-merge Indicates a PR has been approved by a committer. label Jan 2, 2020
@qw4990 qw4990 added the status/LGT2 Indicates that a PR has LGTM 2. label Jan 2, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Jan 2, 2020

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Jan 2, 2020

@lonng merge failed.

@lonng
Copy link
Contributor Author

lonng commented Jan 2, 2020

/run-integration-common-test tidb-test=pr/960

@lonng lonng merged commit 1437d7e into pingcap:master Jan 2, 2020
@lonng lonng deleted the push-down-cast branch January 2, 2020 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/expression status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants