Skip to content
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

Add text IS [NOT] NORMALIZED -> boolean support to SQL and str library #20707

Open
jqnatividad opened this issue Jan 14, 2025 · 1 comment
Open
Labels
enhancement New feature or an improvement of an existing feature

Comments

@jqnatividad
Copy link
Contributor

Description

...as the unicode-normalization crate has the required is_ functions.

https://docs.rs/unicode-normalization/latest/unicode_normalization/index.html#functions

Filing a separate feature request as per @alexander-beedie in #20692 now that #20705 is about to be merged.

Can you split out the request for text IS [NOT] [form] NORMALIZED → boolean as a separate Issue? I took a quick look and it seems it will require an update to the upstream sqlparser-rs lib (which I can likely do, but we'll have to wait for the subsequent release of that lib to add support here). Easier to manage as separate Issues so I can close this one and keep the other on my radar👌

@jqnatividad jqnatividad added the enhancement New feature or an improvement of an existing feature label Jan 14, 2025
@alexander-beedie
Copy link
Collaborator

alexander-beedie commented Jan 15, 2025

FYI: have prepped a PR for sqlparser-rs with parsing support for IS [NOT] [form] NORMALIZED syntax. See: apache/datafusion-sqlparser-rs#1655.

If/when that merges (and we have implemented the expression equivalent) we can look at integration for Polars SQL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or an improvement of an existing feature
Projects
None yet
Development

No branches or pull requests

2 participants