Re: Order changes in PG16 since ICU introduction

Joe Conway <mail@joeconway.com>

From: Joe Conway <mail@joeconway.com>
To: Jeff Davis <pgsql@j-davis.com>, Daniel Verite <daniel@manitou-mail.org>
Cc: Matthias van de Meent <boekewurm+postgres@gmail.com>, Andrew Gierth <andrew@tao11.riddles.org.uk>, Peter Eisentraut <peter.eisentraut@enterprisedb.com>, Sandro Santilli <strk@kbt.io>, Tom Lane <tgl@sss.pgh.pa.us>, Regina Obe <lr@pcorp.us>, pgsql-hackers@lists.postgresql.org
Date: 2023-05-24T16:31:42Z
Lists: pgsql-hackers
On 5/24/23 11:39, Jeff Davis wrote:
> On Mon, 2023-05-22 at 22:09 +0200, Daniel Verite wrote:
>> In practice we're probably getting the "und" ICU locale whereas "fr"
>> would be appropriate.
> 
> This is a good point and illustrates that ICU is not a drop-in
> replacement for libc in all cases.
> 
> I don't see a solution here that doesn't involve some rough edges,
> though. "Locale" is a generic term, and if we continue to insist that
> it really means a libc locale, then ICU will never be on an equal
> footing with libc, let alone the preferred provider.

Huge +1

IMHO the experience should be unified to the degree possible.

-- 
Joe Conway
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com




Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →
  1. Introduce "builtin" collation provider.

  2. ICU: do not convert locale 'C' to 'en-US-u-va-posix'.

  3. initdb: change default --locale-provider back to libc.

  4. CREATE DATABASE: make LOCALE apply to all collation providers.

  5. Remove read-only server settings lc_collate and lc_ctype

  6. ICU: use uloc_getDefault() for initdb.

  7. Doc fixes for commit 1e16af8ab5.

  8. Doc: fix some rendering problems in the PDF docs build.

  9. Doc improvements for language tags and custom ICU collations.

  10. ICU: check for U_STRING_NOT_TERMINATED_WARNING.

  11. Reduce icu_validation_level default to WARNING.

  12. Revert "ICU: do not convert locale 'C' to 'en-US-u-va-posix'."

  13. Fix initdb --no-locale.

  14. Avoid character classification in regex escape parsing.