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

adding locales #123

Open
dkomchenko opened this issue Dec 2, 2024 · 1 comment
Open

adding locales #123

dkomchenko opened this issue Dec 2, 2024 · 1 comment

Comments

@dkomchenko
Copy link

Good afternoon. Why not add all the locales to images by default? Now we have to build CICD only to add a locale. Adding locales to your images right away would save many users from this.

@ardentperf
Copy link

At this point in time, libc locales are discouraged by many people as a default for new databases. A common recommendation is to use the Postgres 17 builtin C collation provider as the database default and to use ICU locales at a table or query level when linguistic ordering is needed.

This is because there are stability issues with linguistic collation (especially libc) and a number of database corruptions have happened in the past related to this. In the context of docker, this means that you can't safely switch from bullseye to bookworm images without logical pg_dump-and-load or logical replication. Switching your container image from bullseye to bookworm and keeping the same PVCs can corrupt your database.

Here's a talk from the postgres development conference earlier this year on the topic https://www.youtube.com/watch?v=KTA6oau7tl8

In consideration of this, I think there's an argument for not adding all locales by default and encouraging usage of the builtin provider and ICU

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

No branches or pull requests

2 participants