From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Improve pg_dump/pg_restore --create --if-exists logic. |
Date: | 2016-11-17 19:59:48 |
Message-ID: | E1c7Sqa-0002hz-CD@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Improve pg_dump/pg_restore --create --if-exists logic.
Teach it not to complain if the dropStmt attached to an archive entry
is actually spelled CREATE OR REPLACE VIEW, since that will happen due to
an upcoming bug fix. Also, if it doesn't recognize a dropStmt, have it
print a WARNING and then emit the dropStmt unmodified. That seems like a
much saner behavior than Assert'ing or dumping core due to a null-pointer
dereference, which is what would happen before :-(.
Back-patch to 9.4 where this option was introduced.
Discussion: <19092(dot)1479325184(at)sss(dot)pgh(dot)pa(dot)us>
Branch
------
REL9_4_STABLE
Details
-------
http://git.postgresql.org/pg/commitdiff/e69b532be797b3fac87a34be71fc97959f8a02aa
Modified Files
--------------
src/bin/pg_dump/pg_backup_archiver.c | 50 ++++++++++++++++++++++++------------
1 file changed, 33 insertions(+), 17 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2016-11-17 20:26:04 | pgsql: Fix pg_dump's handling of circular dependencies in views. |
Previous Message | Tom Lane | 2016-11-17 19:37:06 | pgsql: Re-pgindent src/bin/pg_dump/* |