pgsql: Disallow creating an ICU collation if the DB encoding won't supp - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Disallow creating an ICU collation if the DB encoding won't supp
Date
Msg-id E1mMFyh-0001Jr-3f@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Disallow creating an ICU collation if the DB encoding won't support it.

Previously this was allowed, but the collation effectively vanished
into the ether because of the way lookup_collation() works: you could
not use the collation, nor even drop it.  Seems better to give an
error up front than to leave the user wondering why it doesn't work.

(Because this test is in DefineCollation not CreateCollation, it does
not prevent pg_import_system_collations from creating ICU collations,
regardless of the initially-chosen encoding.)

Per bug #17170 from Andrew Bille.  Back-patch to v10 where ICU support
was added.

Discussion: https://postgr.es/m/17170-95845cf3f0a9c36d@postgresql.org

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/db2760a84191c329c0cdfaa1dae048c32b0c1752

Modified Files
--------------
src/backend/commands/collationcmds.c | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)


pgsql-committers by date:

Previous
From: John Naylor
Date:
Subject: pgsql: Set the volatility of the timestamptz version of date_bin() back
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Set the volatility of the timestamptz version of date_bin() back