Re: ICU for global collation - Mailing list pgsql-hackers

commit f2553d43060edb210b36c63187d52a632448e1d2 says >=1500 in a few places,
but in pg_upgrade says <=1500, which looks wrong for upgrades from v15.
I think it should say <= 1400.

On Wed, Feb 02, 2022 at 02:01:23PM +0100, Peter Eisentraut wrote:
> --- a/src/bin/pg_dump/pg_dump.c
> +++ b/src/bin/pg_dump/pg_dump.c
> @@ -2792,6 +2794,10 @@ dumpDatabase(Archive *fout)
>          appendPQExpBuffer(dbQry, "datminmxid, ");
>      else
>          appendPQExpBuffer(dbQry, "0 AS datminmxid, ");
> +    if (fout->remoteVersion >= 150000)
> +        appendPQExpBuffer(dbQry, "datcollprovider, ");
> +    else
> +        appendPQExpBuffer(dbQry, "'c' AS datcollprovider, ");
>      appendPQExpBuffer(dbQry,
>                        "(SELECT spcname FROM pg_tablespace t WHERE t.oid = dattablespace) AS tablespace, "
>                        "shobj_description(oid, 'pg_database') AS description "

> diff --git a/src/bin/pg_upgrade/info.c b/src/bin/pg_upgrade/info.c
> index 69ef23119f..2a9ca0e389 100644
> --- a/src/bin/pg_upgrade/info.c
> +++ b/src/bin/pg_upgrade/info.c
> @@ -312,11 +312,20 @@ get_db_infos(ClusterInfo *cluster)
>                  i_spclocation;
>      char        query[QUERY_ALLOC];
>  
>      snprintf(query, sizeof(query),
> -             "SELECT d.oid, d.datname, d.encoding, d.datcollate, d.datctype, "
> +             "SELECT d.oid, d.datname, d.encoding, d.datcollate, d.datctype, ");
> +    if (GET_MAJOR_VERSION(old_cluster.major_version) <= 1500)
> +        snprintf(query + strlen(query), sizeof(query) - strlen(query),
> +                 "'c' AS datcollprovider, NULL AS daticucoll, ");
> +    else
> +        snprintf(query + strlen(query), sizeof(query) - strlen(query),
> +                 "datcollprovider, daticucoll, ");
> +    snprintf(query + strlen(query), sizeof(query) - strlen(query),
>               "pg_catalog.pg_tablespace_location(t.oid) AS spclocation "
>               "FROM pg_catalog.pg_database d "
>               " LEFT OUTER JOIN pg_catalog.pg_tablespace t "

> --- a/src/bin/psql/describe.c
> +++ b/src/bin/psql/describe.c
> @@ -896,6 +896,18 @@ listAllDbs(const char *pattern, bool verbose)
>                        gettext_noop("Encoding"),
>                        gettext_noop("Collate"),
>                        gettext_noop("Ctype"));
> +    if (pset.sversion >= 150000)
> +        appendPQExpBuffer(&buf,
> +                          "       d.daticucoll as \"%s\",\n"
> +                          "       CASE d.datcollprovider WHEN 'c' THEN 'libc' WHEN 'i' THEN 'icu' END AS
\"%s\",\n",
> +                          gettext_noop("ICU Collation"),
> +                          gettext_noop("Coll. Provider"));
> +    else
> +        appendPQExpBuffer(&buf,
> +                          "       d.datcollate as \"%s\",\n"
> +                          "       'libc' AS \"%s\",\n",
> +                          gettext_noop("ICU Collation"),
> +                          gettext_noop("Coll. Provider"));
>      appendPQExpBufferStr(&buf, "       ");
>      printACLColumn(&buf, "d.datacl");
>      if (verbose)

> @@ -4617,6 +4629,15 @@ listCollations(const char *pattern, bool verbose, bool showSystem)
>                        gettext_noop("Collate"),
>                        gettext_noop("Ctype"));
>  
> +    if (pset.sversion >= 150000)
> +        appendPQExpBuffer(&buf,
> +                          ",\n       c.collicucoll AS \"%s\"",
> +                          gettext_noop("ICU Collation"));
> +    else
> +        appendPQExpBuffer(&buf,
> +                          ",\n       c.collcollate AS \"%s\"",
> +                          gettext_noop("ICU Collation"));
> +
>      if (pset.sversion >= 100000)
>          appendPQExpBuffer(&buf,
>                            ",\n       CASE c.collprovider WHEN 'd' THEN 'default' WHEN 'c' THEN 'libc' WHEN 'i' THEN
'icu'END AS \"%s\"",