pgsql: Support --no-comments in pg_dump, pg_dumpall, pg_restore. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Support --no-comments in pg_dump, pg_dumpall, pg_restore.
Date
Msg-id E1eeo7N-0003ZA-Jo@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Support --no-comments in pg_dump, pg_dumpall, pg_restore.

We have switches already to suppress other subsidiary object properties,
such as ACLs, security labels, ownership, and tablespaces, so just on
the grounds of symmetry we should allow suppressing comments as well.
Also, commit 0d4e6ed30 added a positive reason to have this feature,
i.e. to allow obtaining the old behavior of selective pg_restore should
anyone desire that.

Recent commits have removed the cases where pg_dump emitted comments on
built-in objects that the restoring user might not have privileges to
comment on, so the original primary motivation for this feature is gone,
but it still seems at least somewhat useful in its own right.

Robins Tharakan, reviewed by Fabrízio Mello

Discussion: https://postgr.es/m/CAEP4nAx22Z4ch74oJGzr5RyyjcyUSbpiFLyeYXX8pehfou92ug@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/1368e92e16a098338e39c8e540bdf9f6cf35ebf4

Modified Files
--------------
doc/src/sgml/ref/pg_dump.sgml        |  9 +++++++++
doc/src/sgml/ref/pg_dumpall.sgml     |  9 +++++++++
doc/src/sgml/ref/pg_restore.sgml     |  9 +++++++++
src/bin/pg_dump/pg_backup.h          |  2 ++
src/bin/pg_dump/pg_backup_archiver.c |  6 +++++-
src/bin/pg_dump/pg_dump.c            | 19 +++++++++++++++++--
src/bin/pg_dump/pg_dumpall.c         |  9 +++++++--
src/bin/pg_dump/pg_restore.c         |  4 ++++
8 files changed, 62 insertions(+), 5 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Add missing "static" markers.
Next
From: Bruce Momjian
Date:
Subject: pgsql: Fix C comment typo