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-policies.
Date: Tue, 16 Sep 2025 02:55:51 +0000
Message-ID: <E1uyLrD-000vlg-1S@gemulon.postgresql.org> (raw)

pg_restore: Fix comment handling with --no-policies.

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

This commit fixes the issue by ensuring that comments on policies
are also skipped when --no-policies is used.

Backpatch to v18, where --no-policies was added in pg_restore.

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: 18

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/66dabc06b1b0aaadfd5be640a2e3efbeea0fcaf5

Modified Files
--------------
src/bin/pg_dump/pg_backup_archiver.c |  7 +++++-
src/bin/pg_dump/t/002_pg_dump.pl     | 46 ++++++++++++++++++++++++++++++++++++
2 files changed, 52 insertions(+), 1 deletion(-)



view thread (9+ messages)  latest in thread

Message-ID: <E1uyLrD-000vlg-1S@gemulon.postgresql.org>
Permalink:  ../E1uyLrD-000vlg-1S@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1uyLrD-000vlg-1S@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-policies.
  In-Reply-To: <E1uyLrD-000vlg-1S@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