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

ranger: check the collation when checking like func (#16380) #16783

Merged
merged 2 commits into from
Apr 26, 2020

Conversation

sre-bot
Copy link
Contributor

@sre-bot sre-bot commented Apr 24, 2020

cherry-pick #16380 to release-4.0


What problem does this PR solve?

Issue Number: close #15988

Problem Summary:

We didn't check the LIKE func's collation when checking whether it can be used to build range.

What is changed and how it works?

What's Changed:

Check the collation info, and if the LIKE's collation info is different with the column's.
Don't use it to build range.

Related changes

  • Need to cherry-pick to the release branch

Check List

Tests

  • Unit test

@sre-bot sre-bot requested a review from a team as a code owner April 24, 2020 05:18
@sre-bot
Copy link
Contributor Author

sre-bot commented Apr 24, 2020

/run-all-tests

Copy link
Member

@wjhuang2016 wjhuang2016 left a comment

Choose a reason for hiding this comment

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

LGTM

@winoros
Copy link
Member

winoros commented Apr 24, 2020

/run-integration-copr-test

1 similar comment
@winoros
Copy link
Member

winoros commented Apr 26, 2020

/run-integration-copr-test

Copy link
Member

@bb7133 bb7133 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
Member

@bb7133 bb7133 left a comment

Choose a reason for hiding this comment

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

LGTM

@SunRunAway SunRunAway added the status/can-merge Indicates a PR has been approved by a committer. label Apr 26, 2020
@sre-bot
Copy link
Contributor Author

sre-bot commented Apr 26, 2020

/run-all-tests

@sre-bot sre-bot merged commit bbe6475 into pingcap:release-4.0 Apr 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sig/planner SIG: Planner status/can-merge Indicates a PR has been approved by a committer. type/4.0-cherry-pick
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants