Re: Collation versioning

From: Julien Rouhaud <rjuju123(at)gmail(dot)com>
To: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
Cc: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>, Douglas Doole <dougdoole(at)gmail(dot)com>, Christoph Berg <myon(at)debian(dot)org>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Collation versioning
Date: 2020-02-27 07:45:35
Message-ID: 20200227074535.4qmrehctqi6nrqei@nol
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Feb 27, 2020 at 04:10:14PM +1300, Thomas Munro wrote:
> On Thu, Feb 27, 2020 at 3:29 AM Julien Rouhaud <rjuju123(at)gmail(dot)com> wrote:
> > [v10]
>
> Thanks. I'll do some more testing and review soon. It'd be really
> cool to get this into PG13.

Thanks!

>
> FYI cfbot said:
>
> +++ /home/travis/build/postgresql-cfbot/postgresql/src/test/regress/results/collate.icu.utf8.out
> 2020-02-26 14:45:52.114401999 +0000
> ...
> - icuidx06_d_en_fr_ga | "default" | up to date
> + icuidx06_d_en_fr_ga | "default" | out of date

Oh. It turns out that pg_collation_actual_version() isn't handling the
default collation:

# SELECT pg_collation_actual_version(100), pg_collation_actual_version(c.oid)
FROM pg_database d
JOIN pg_collation c on c.collname = d.datcollate
WHERE datname = current_database();
pg_collation_actual_version | pg_collation_actual_version
-----------------------------+-----------------------------
<NULL> | 2.30
(1 row)

Fixed in v11 by changing pg_collation_actual_version() to handle default
collation too, as it seems a better behavior.

Attachment Content-Type Size
0001-Remove-pg_collation.collversion-v11.patch text/x-diff 23.4 KB
0002-Add-pg_depend.refobjversion-v11.patch text/x-diff 11.6 KB
0003-Implement-type-regcollation-v11.patch text/x-diff 8.9 KB
0004-Track-collation-versions-for-indexes-v11.patch text/x-diff 61.5 KB
0005-Preserve-index-dependencies-on-collation-during-pg_u-v11.patch text/x-diff 38.9 KB
0006-Add-a-new-ALTER-INDEX-name-ALTER-COLLATION-name-REFR-v11.patch text/x-diff 14.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Kyotaro Horiguchi 2020-02-27 08:05:30 Re: Crash by targetted recovery
Previous Message Michael Paquier 2020-02-27 07:32:11 Re: reindex concurrently and two toast indexes