-
Notifications
You must be signed in to change notification settings - Fork 209
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
{171428666}: column character encoding #4366
base: main
Are you sure you want to change the base?
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.
Lint check:
@@ sqlite/src/parse.y @@
+ Custom tokens and rules must be defined after SQLite's.
+ Please update SQL documentation and bubble diagrams to reflect the changes you've made, if necessary.
e36af9f
to
1b7f975
Compare
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.
Coding style check: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Error ⚠.
Regression testing: 11/528 tests failed ⚠.
The first 10 failing tests are:
lostwrite [setup failure]
schemalk
auth_twofiles_generated
auth_queueodh_generated
auth
comdb2sys
comdb2sys_queueodh_generated
comdb2sys_pagesize_generated
sc_downgrade
truncatesc_offline_generated
1b7f975
to
3f45643
Compare
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.
Lint check:
@@ sqlite/src/parse.y @@
+ Custom tokens and rules must be defined after SQLite's.
+ Please update SQL documentation and bubble diagrams to reflect the changes you've made, if necessary.
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.
Coding style check: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 6/529 tests failed ⚠.
The first 10 failing tests are:
schemalk_logicalsc_generated
schemalk_extralock_generated
schemalk
queuedb_rollover
truncatesc_offline_generated
A new `ENCODING` keyword is introduced in this patch, that specifies a character set for a column. Invalid character encoding will be rejected. So far only `"utf8"` and `NONE` are supported. This allows users to conveniently create an indexable utf8 cstring column. It's implemented as a check constraint using the `utf8_validate()` function. Signed-off-by: Rivers Zhang <hzhang320@bloomberg.net>
3f45643
to
f532fce
Compare
A new
ENCODING
keyword is introduced in this patch, that specifies a character set for a column. Invalid character encoding will be rejected. So far only"utf8"
andNONE
are supported.This allows users to conveniently create an indexable utf8 cstring column. It's implemented as a check constraint using the
utf8_validate()
function.