From: Tom Lane <tgl@sss.pgh.pa.us>
To: Jeff Davis <pgsql@j-davis.com>
Cc: Daniel Verite <daniel@manitou-mail.org>
Cc: Andrew Gierth <andrew@tao11.riddles.org.uk>
Cc: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
Cc: Sandro Santilli <strk@kbt.io>
Cc: Regina Obe <lr@pcorp.us>
Cc: pgsql-hackers@lists.postgresql.org
Subject: Re: Order changes in PG16 since ICU introduction
Date: Mon, 08 May 2023 17:47:55 -0400
Message-ID: <483826.1683582475@sss.pgh.pa.us> (raw)
In-Reply-To: <37520ec1ae9591f83132f82dbd625f3fc2d69c16.camel@j-davis.com>
References: <543e4ce6-8205-4aea-b54b-12b9ef747bf6@manitou-mail.org>
<654a49f7ff7461bcf47be4181430678d45f93858.camel@j-davis.com>
<37520ec1ae9591f83132f82dbd625f3fc2d69c16.camel@j-davis.com>
Jeff Davis <pgsql@j-davis.com> writes:
> === 0001: do not convert C to en-US-u-va-posix
> I plan to commit this soon.
Several buildfarm animals have failed since this went in. The
only one showing enough info to diagnose is siskin [1]:
@@ -1043,16 +1043,15 @@
ERROR: ICU locale "nonsense-nowhere" has unknown language "nonsense"
HINT: To disable ICU locale validation, set parameter icu_validation_level to DISABLED.
CREATE COLLATION testx (provider = icu, locale = 'C'); -- fails
-ERROR: could not convert locale name "C" to language tag: U_ILLEGAL_ARGUMENT_ERROR
+NOTICE: using standard form "en-US-u-va-posix" for locale "C"
CREATE COLLATION testx (provider = icu, locale = '@colStrength=primary;nonsense=yes'); -- fails
ERROR: could not convert locale name "@colStrength=primary;nonsense=yes" to language tag: U_ILLEGAL_ARGUMENT_ERROR
SET icu_validation_level = WARNING;
CREATE COLLATION testx (provider = icu, locale = '@colStrength=primary;nonsense=yes'); DROP COLLATION testx;
WARNING: could not convert locale name "@colStrength=primary;nonsense=yes" to language tag: U_ILLEGAL_ARGUMENT_ERROR
+ERROR: collation "testx" already exists
CREATE COLLATION testx (provider = icu, locale = 'C'); DROP COLLATION testx;
-WARNING: could not convert locale name "C" to language tag: U_ILLEGAL_ARGUMENT_ERROR
-WARNING: ICU locale "C" has unknown language "c"
-HINT: To disable ICU locale validation, set parameter icu_validation_level to DISABLED.
+NOTICE: using standard form "en-US-u-va-posix" for locale "C"
CREATE COLLATION testx (provider = icu, locale = 'nonsense-nowhere'); DROP COLLATION testx;
WARNING: ICU locale "nonsense-nowhere" has unknown language "nonsense"
HINT: To disable ICU locale validation, set parameter icu_validation_level to DISABLED.
I suppose this is environment-dependent. Sadly, the buildfarm
client does not show the prevailing LANG or LC_XXX settings.
regards, tom lane
[1] https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=siskin&dt=2023-05-08%2020%3A09%3A26
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: tgl@sss.pgh.pa.us, pgsql@j-davis.com, daniel@manitou-mail.org, andrew@tao11.riddles.org.uk, peter.eisentraut@enterprisedb.com, strk@kbt.io, lr@pcorp.us, pgsql-hackers@lists.postgresql.org
Subject: Re: Order changes in PG16 since ICU introduction
In-Reply-To: <483826.1683582475@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox