Re: pgsql: pg_upgrade: Preserve relfilenodes and tablespace OIDs.

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Christoph Berg <myon(at)debian(dot)org>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: pgsql: pg_upgrade: Preserve relfilenodes and tablespace OIDs.
Date: 2022-02-17 15:59:37
Message-ID: CA+Tgmoad8ZiMchVvD743GuETBozmkjVGv-7r5cKm7FoMxthODA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Sun, Feb 13, 2022 at 6:51 AM Christoph Berg <myon(at)debian(dot)org> wrote:
> Re: Robert Haas
> > pg_upgrade: Preserve relfilenodes and tablespace OIDs.
>
> > src/bin/pg_dump/pg_dumpall.c | 3 +
>
> --- a/src/bin/pg_dump/pg_dumpall.c
> +++ b/src/bin/pg_dump/pg_dumpall.c
> @@ -1066,6 +1066,9 @@ dumpTablespaces(PGconn *conn)
> /* needed for buildACLCommands() */
> fspcname = pg_strdup(fmtId(spcname));
>
> + appendPQExpBufferStr(buf, "\n-- For binary upgrade, must preserve pg_table
> + appendPQExpBuffer(buf, "SELECT pg_catalog.binary_upgrade_set_next_pg_table
>
> This needs to be guarded with "if (binary_upgrade)".

Right. Sorry about that, and sorry for not responding sooner also. Fix
pushed now.

--
Robert Haas
EDB: http://www.enterprisedb.com

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2022-02-17 20:03:47 pgsql: Avoid dangling-pointer usage in pg_basebackup progress reports.
Previous Message Robert Haas 2022-02-17 15:58:33 pgsql: Add missing binary-upgrade guard.

Browse pgsql-hackers by date

  From Date Subject
Next Message Fujii Masao 2022-02-17 16:46:38 Re: postgres_fdw: commit remote (sub)transactions in parallel during pre-commit
Previous Message Daniel Gustafsson 2022-02-17 15:42:22 Re: [PATCH] Fix out-of-bouds access (src/common/wchar.c)