From: Jeff Davis <pgsql@j-davis.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Andrew Dunstan <andrew@dunslane.net>
Cc: Peter Eisentraut <peter@eisentraut.org>
Cc: pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Add standard collation UNICODE
Date: Mon, 13 Mar 2023 12:40:28 -0700
Message-ID: <0c6fa66f2753217d2a40480a96bd2ccf023536a1.camel@j-davis.com> (raw)
In-Reply-To: <1253026.1678567661@sss.pgh.pa.us>
References: <E1pac4x-002qVH-Ce@gemulon.postgresql.org>
<582e43d0a814e09c62ea662dd851f43bfc137a16.camel@j-davis.com>
<c05d7d71-e869-f6ae-b83c-2d5cab0a0dc4@dunslane.net>
<231acf388c37597a9f0f8a3af9a899df99c28e1d.camel@j-davis.com>
<1253026.1678567661@sss.pgh.pa.us>
On Sat, 2023-03-11 at 15:47 -0500, Tom Lane wrote:
> In general, I see no good reason for our regression tests to be
> making
> assumptions about exactly how ICU's root locale behaves, so I'd
> suggest
> just lobotomizing this test case so it doesn't depend on upper/lower
> sort order.
[ Looks like Peter already found the answer, but I had some additional
commentary so I'll send this anyway. ]
Mystery solved: "und" was not a valid spelling of "root" until version
55. So it fell back to the default locale, which is sensitive to the
environment.
With this in mind, I think we should take the opposite lesson and keep
this test as-is. We don't have to promise the behavior in the docs, but
it did catch a real issue. In fact, it caught a dangerous issue,
because a change to lc_messages could cause a corrupt index.
I'd like to see if there's a way to reliably detect that a fallback to
the environment-sensitive default locale has happened, and throw an
error. I don't think that happens in versions 55 and later, but it
would be nice to have a test because earlier versions of ICU seem
dangerous to me.
Regards,
Jeff Davis
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-committers@postgresql.org
Cc: pgsql@j-davis.com, tgl@sss.pgh.pa.us, andrew@dunslane.net, peter@eisentraut.org, pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Add standard collation UNICODE
In-Reply-To: <0c6fa66f2753217d2a40480a96bd2ccf023536a1.camel@j-davis.com>
* 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