pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Fujii Masao <fujii@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: pg_restore: Fix comment handling with --no-publications / --no-s
Date: Tue, 16 Sep 2025 01:42:22 +0000
Message-ID: <E1uyKi5-000vLZ-3A@gemulon.postgresql.org> (raw)

pg_restore: Fix comment handling with --no-publications / --no-subscriptions.

Previously, pg_restore did not skip comments on publications or subscriptions
even when --no-publications or --no-subscriptions was specified. As a result,
it could issue COMMENT commands for objects that were never created,
causing those commands to fail.

This commit fixes the issue by ensuring that comments on publications and
subscriptions are also skipped when the corresponding options are used.

Backpatch to all supported versions.

Author: Jian He <jian.universality@gmail.com>
Co-authored-by: Fujii Masao <masao.fujii@gmail.com>
Discussion: https://postgr.es/m/CACJufxHCt00pR9h51AVu6+yPD5J7JQn=7dQXxqacj0XyDhc-fA@mail.gmail.com
Backpatch-through: 13

Branch
------
REL_14_STABLE

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

Modified Files
--------------
src/bin/pg_dump/pg_backup_archiver.c | 14 ++++++++++++++
src/bin/pg_dump/t/002_pg_dump.pl     | 37 ++++++++++++++++++++++++++++++++++++
2 files changed, 51 insertions(+)



view thread (9+ messages)  latest in thread

Message-ID: <E1uyKi5-000vLZ-3A@gemulon.postgresql.org>
Permalink:  ../E1uyKi5-000vLZ-3A@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1uyKi5-000vLZ-3A@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: fujii@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: pg_restore: Fix comment handling with --no-publications / --no-s
  In-Reply-To: <E1uyKi5-000vLZ-3A@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox