-
Notifications
You must be signed in to change notification settings - Fork 4.1k
sql: consolidate the handling of strings and collated strings #29001
Copy link
Copy link
Closed
Labels
A-sql-datatypesSQL column types usable in table descriptors.SQL column types usable in table descriptors.C-cleanupTech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.X-staleno-issue-activity
Description
Suggested by @BramGruneir in #28951 (review):
Also, is there a reason to not consolidate collated strings with strings now? Can the absence of the locale be enough to make this practical? It looks like most of their handling is repeated code.
(looking at the code in sql/sqlbase)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-sql-datatypesSQL column types usable in table descriptors.SQL column types usable in table descriptors.C-cleanupTech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.X-staleno-issue-activity