-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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: fix the issue that incorrect result for a predicate that uses the CHAR() function (#16014) #16559
expression: fix the issue that incorrect result for a predicate that uses the CHAR() function (#16014) #16559
Conversation
/run-all-tests |
wait for (tikv#7548)(tikv/tikv#7548) |
/run-all-tests |
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
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
@sre-bot merge failed. |
/run-integration-copr-test |
1 similar comment
/run-integration-copr-test |
/run-all-tests |
/run-all-tests |
/run-integration-copr-test tikv=pr/7548 |
/run-integration-copr-test tikv=pr/7548 |
1 similar comment
/run-integration-copr-test tikv=pr/7548 |
/run-all-tests tikv=pr/7548 |
/run-all-tests tikv=pr/7548 |
wait for #16446 cherry pick |
/run-all-tests |
/run-all-tests tikv=pr/7548 |
/run-all-tests |
/merge |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
cherry-pick #16014 to release-4.0
What problem does this PR solve?
Issue Number: close #15986
What is changed and how it works?
When we use the
string
type to determine thebool
value, thestring
type is converted to afloat
type instead of anint
type.Check List
Tests