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: push down expr json_length to tiflash #17588

Merged
merged 3 commits into from
Jun 2, 2020

Conversation

solotzg
Copy link
Contributor

@solotzg solotzg commented Jun 2, 2020

Signed-off-by: Tong Zhigao tongzhigao@pingcap.com

What problem does this PR solve?

What is changed and how it works?

What's Changed:

  • Add json_length to white list which can be pushed down to tiflash.

Check List

Tests

  • Manual test (add detailed scripts or steps below)

Release note

  • No release note

Signed-off-by: Tong Zhigao <tongzhigao@pingcap.com>
@sre-bot
Copy link
Contributor

sre-bot commented Jun 2, 2020

@sre-bot
Copy link
Contributor

sre-bot commented Jun 2, 2020

@solotzg
Copy link
Contributor Author

solotzg commented Jun 2, 2020

/run-all-tests

@solotzg
Copy link
Contributor Author

solotzg commented Jun 2, 2020

/rebuild

Copy link
Contributor

@lzmhhh123 lzmhhh123 left a comment

Choose a reason for hiding this comment

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

Please add a unit test.

Signed-off-by: Tong Zhigao <tongzhigao@pingcap.com>
@solotzg
Copy link
Contributor Author

solotzg commented Jun 2, 2020

@lzmhhh123 PTAL

Copy link
Contributor

@lzmhhh123 lzmhhh123 left a comment

Choose a reason for hiding this comment

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

LGTM

@lzmhhh123 lzmhhh123 added the status/LGT1 Indicates that a PR has LGTM 1. label Jun 2, 2020
@lzmhhh123 lzmhhh123 requested a review from wshwsh12 June 2, 2020 09:19
@codecov
Copy link

codecov bot commented Jun 2, 2020

Codecov Report

Merging #17588 into master will decrease coverage by 0.0836%.
The diff coverage is 50.0000%.

@@               Coverage Diff                @@
##             master     #17588        +/-   ##
================================================
- Coverage   79.5052%   79.4216%   -0.0837%     
================================================
  Files           520        520                
  Lines        140041     139904       -137     
================================================
- Hits         111340     111114       -226     
- Misses        19740      19821        +81     
- Partials       8961       8969         +8     

Copy link
Contributor

@wshwsh12 wshwsh12 left a comment

Choose a reason for hiding this comment

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

LGTM

@wshwsh12 wshwsh12 added status/LGT2 Indicates that a PR has LGTM 2. status/can-merge Indicates a PR has been approved by a committer. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Jun 2, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Jun 2, 2020

/run-all-tests

@sre-bot sre-bot merged commit f03677c into pingcap:master Jun 2, 2020
@solotzg solotzg deleted the json-len branch June 2, 2020 09:37
@sre-bot
Copy link
Contributor

sre-bot commented Jun 2, 2020

cherry pick to release-4.0 failed

@you06
Copy link
Contributor

you06 commented Jun 2, 2020

/unlabel component/expression

@you06
Copy link
Contributor

you06 commented Jun 2, 2020

/label component/expression

@you06
Copy link
Contributor

you06 commented Jun 2, 2020

/run-cherry-picker

sre-bot pushed a commit to sre-bot/tidb that referenced this pull request Jun 2, 2020
Signed-off-by: sre-bot <sre-bot@pingcap.com>
@sre-bot
Copy link
Contributor

sre-bot commented Jun 2, 2020

cherry pick to release-4.0 in PR #17592

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