-
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: set collation id to negative in tipb if new collations are enabled #14883
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
2d561a3
to
8207c1b
Compare
/run-all-tests |
Codecov Report
@@ Coverage Diff @@
## master #14883 +/- ##
===========================================
Coverage ? 80.2286%
===========================================
Files ? 503
Lines ? 130957
Branches ? 0
===========================================
Hits ? 105065
Misses ? 17588
Partials ? 8304 |
/run-check_dev_2 |
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 |
@bb7133 merge failed. |
What problem does this PR solve?
This is a trick for the new collation implementations:
components of the cluster(for example, TiKV) is able to aware of it without any change to
the protocol definition.
_bin
collations without padding.What is changed and how it works?
Check List
Tests
Code changes
Side effects
Related changes
Release note