pgsql-server: Here is another patch that fixes a stack of pg_dump bugs: - Mailing list pgsql-committers

From momjian@svr1.postgresql.org (Bruce Momjian)
Subject pgsql-server: Here is another patch that fixes a stack of pg_dump bugs:
Date
Msg-id 20040719213948.B6760D1B182@svr1.postgresql.org
Whole thread Raw
List pgsql-committers
Log Message:
-----------
Here is another patch that fixes a stack of pg_dump bugs:

* Fix help text ordering

* Add back --set-session-authorization to pg_dumpall.  Updated the docs
for that.  Updated help for that.

* Dump ALTER USER commands for the cluster owner ("pgsql").  These are
dumped AFTER the create user and create database commands in case the
permissions to do these have been revoked.

* Dump ALTER OWNER for public schema (because it's possible to change
it).  This was done by adding TOC entries for the public schema, and
filtering them out at archiver time.  I also save the owner in the TOC
entry just for the public schema.

* Suppress dumping single quotes around schema_path and DateStyle
options when they are set using ALTER USER or ALTER DATABASE.  Added a
comment to the steps in guc.c to remind people to update that list.

* Fix dumping in --clean mode against a pre-7.3 server.  It just sets
all drop statements to assume the public schema, allowing it to restore
without error.

* Cleaned up text output.  eg. Don't output -- Tablespaces comment if
there are none.  Same for groups and users.

* Make the commands to DELETE FROM pg_shadow and DELETE FROM pg_group
only be output when -c mode is enabled.  I'm not sure why that hasn't
been done before?!?!

This should be good for application asap, after which I will start on
regression dumping 7.0-7.4 databases.

Christopher Kings-Lynne

Modified Files:
--------------
    pgsql-server/doc/src/sgml/ref:
        pg_dumpall.sgml (r1.45 -> r1.46)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql-server/doc/src/sgml/ref/pg_dumpall.sgml.diff?r1=1.45&r2=1.46)
    pgsql-server/src/backend/utils/misc:
        guc.c (r1.220 -> r1.221)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql-server/src/backend/utils/misc/guc.c.diff?r1=1.220&r2=1.221)
    pgsql-server/src/bin/pg_dump:
        pg_backup_archiver.c (r1.89 -> r1.90)

(http://developer.postgresql.org/cvsweb.cgi/pgsql-server/src/bin/pg_dump/pg_backup_archiver.c.diff?r1=1.89&r2=1.90)
        pg_dump.c (r1.379 -> r1.380)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql-server/src/bin/pg_dump/pg_dump.c.diff?r1=1.379&r2=1.380)
        pg_dumpall.c (r1.44 -> r1.45)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql-server/src/bin/pg_dump/pg_dumpall.c.diff?r1=1.44&r2=1.45)

pgsql-committers by date:

Previous
From: tgl@svr1.postgresql.org (Tom Lane)
Date:
Subject: pgsql-server: Fix incorrect tracking of session authorization in the
Next
From: Christopher Kings-Lynne
Date:
Subject: Re: pgsql-server: Fix incorrect tracking of session