Re: Crash report for some ICU-52 (debian8) COLLATE and work_mem values
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Peter Geoghegan <pg@bowt.ie>
Cc: Robert Haas <robertmhaas@gmail.com>,
Peter Eisentraut <peter.eisentraut@2ndquadrant.com>,
Daniel Verite <daniel@manitou-mail.org>,
PostgreSQL mailing lists <pgsql-bugs@postgresql.org>
Date: 2017-08-09T18:07:48Z
Lists: pgsql-bugs, pgsql-hackers
Peter Geoghegan <pg@bowt.ie> writes: > On Wed, Aug 9, 2017 at 10:35 AM, Robert Haas <robertmhaas@gmail.com> wrote: >> In other words, excluding, say, emoji collations from what gets >> imported is just making a value judgement that those collations aren't >> important and people shouldn't want to use them. > Yes, it is. I think that's fine, though. Other database systems that > use ICU for collations do this. Without exception, I think. Actually, I don't think that's the issue at all. People are free to make other ICU collations if they want to. My point is that we should encourage them to do that, rather than depend on initdb-provided collations, because manually-created collations are much more certain to move across version upgrades safely. If we were sure that pg_import_system_collations would produce pretty much the same set of collation names with future ICU releases as it does with current ones, then there would be no issue --- but the evidence at hand suggests the opposite. I want to do something to address that stability issue before it comes back to bite us. I suppose a different way to address this would be to make pg_upgrade smart enough to deal with the situation, by creating ICU collations that are used in the source installation but are missing from the initdb-provided set in the target. But even if we had that, I'm dubious that having hundreds of collations present by default is really all that user-friendly. regards, tom lane
Commits
-
Reject use of ucol_strcollUTF8() before ICU 53
- d6391b03b302 10.0 landed
-
Rethink behavior of pg_import_system_collations().
- 0b13b2a7712b 10.0 cited
-
Faster expression evaluation and targetlist projection.
- b8d7f053c5c2 10.0 cited