-
Notifications
You must be signed in to change notification settings - Fork 605
Add support for IS [NOT] [form] NORMALIZED
#1655
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
Conversation
1ff1908
to
c36a23b
Compare
c36a23b
to
b24b958
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.
Thank you @alexander-beedie -- I think this looks quite nice and well tested
fyi @iffyio
@@ -1118,7 +1124,7 @@ impl fmt::Display for LambdaFunction { | |||
/// `OneOrManyWithParens` implements `Deref<Target = [T]>` and `IntoIterator`, | |||
/// so you can call slice methods on it and iterate over items | |||
/// # Examples | |||
/// Acessing as a slice: | |||
/// Accessing as a slice: |
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.
Thank you for these cleanups
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.
Thanks @alexander-beedie!
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.
Just one comment regarding the API signature, otherwise this looks good to me!
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! Thanks @alexander-beedie!
…o escape-literals * 'main' of github.com:hansott/datafusion-sqlparser-rs: National strings: check if dialect supports backslash escape (apache#1672) Add support for Create Iceberg Table statement for Snowflake parser (apache#1664) Add support for Snowflake account privileges (apache#1666) Update rat_exclude_file.txt (apache#1670) Update verson to 0.54.0 and update changelog (apache#1668) Add support for Snowflake AT/BEFORE (apache#1667) Add support for qualified column names in JOIN ... USING (apache#1663) Add support for `IS [NOT] [form] NORMALIZED` (apache#1655) fix parsing of `INSERT INTO ... SELECT ... RETURNING ` (apache#1661) Add support for Snowflake column aliases that use SQL keywords (apache#1632)
Co-authored-by: Alexander Beedie <[email protected]>
Co-authored-by: Alexander Beedie <[email protected]>
This reverts commit 2741fd0.
Co-authored-by: Alexander Beedie <[email protected]>
Adds parsing support for
IS [NOT] [<form>] NORMALIZED → bool
syntax:Details from the PostgreSQL string function docs:
https://www.postgresql.org/docs/current/functions-string.html
NFC
: Canonical Decomposition, followed by Canonical Composition.NFD
: Canonical Decomposition.NFKC
: Compatibility Decomposition, followed by Canonical Composition.NFKD
: Compatibility Decomposition.As the normalised forms are fixed (there are only these four), it seemed reasonable to return the parsed form as a new
Option<NormalizationForm>
Enum (which helps the caller as they don't have to check the string or case-normalise it, and can then jump straight into some associated match block, etc).(Also: fixed a few minor typos).
Examples
Default/omitted
form
:Specific
form
: