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

release-22.2: collatedstring: support C and POSIX in expressions #97415

Merged
merged 2 commits into from
Feb 22, 2023

Conversation

rafiss
Copy link
Collaborator

@rafiss rafiss commented Feb 21, 2023

Backport 2/2 commits from #96828.

/cc @cockroachdb/release

Release justification: This blocks support for SQLAlchemy with psycopg3


fixes #50734
fixes #95667
informs #57255


collatedstring: create new package

Move the small amount of code from tree/collatedstring.go


collatedstring: support C and POSIX in expressions

Release note (sql change): Expressions of the form COLLATE "default",
COLLATE "C", and COLLATE "POSIX" are now supported. Since the
default collation cannot be changed currently, these expressions are all
equivalent. The expressions are evaluated by treating the input as a
normal string, and ignoring the collation.

This means that comparisons between strings and collated strings that
use "default", "C", or "POSIX" are now supported.

Creating a column with the "C" or "POSIX" collations is still not
supported.

Move the small amount of code from tree/collatedstring.go

Release note: None
@rafiss rafiss requested review from otan and a team February 21, 2023 21:13
@rafiss rafiss requested a review from a team as a code owner February 21, 2023 21:13
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@rafiss rafiss changed the title collatedstring: create new package release-22.2: collatedstring: support C and POSIX in expressions Feb 21, 2023
Release note (sql change): Expressions of the form `COLLATE "default"`,
`COLLATE "C"`, and `COLLATE "POSIX"` are now supported. Since the
default collation cannot be changed currently, these expressions are all
equivalent. The expressions are evaluated by treating the input as a
normal string, and ignoring the collation.

This means that comparisons between strings and collated strings that
use "default", "C", or "POSIX" are now supported.

Creating a column with the "C" or "POSIX" collations is still not
supported.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants