-
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: return binary
charset and collation if there is no string-type argument in DeriveCollationFromExprs
#15250
Conversation
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, Please fix the ci
Codecov Report
@@ Coverage Diff @@
## master #15250 +/- ##
===========================================
Coverage 80.5354% 80.5354%
===========================================
Files 503 503
Lines 134398 134398
===========================================
Hits 108238 108238
Misses 17685 17685
Partials 8475 8475 |
Fixed, PTAL @bb7133 @wjhuang2016 |
/run-unit-test |
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 |
@qw4990 merge failed. |
/run-unit-test |
What problem does this PR solve?
Update the function
DeriveCollationFromExprs
to let it returnbinary
charset and collation if there is no string-type argument.Check List
Tests