Improve pg_dump/pg_restore --create --if-exists logic.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 17 Nov 2016 19:59:13 +0000 (14:59 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 17 Nov 2016 19:59:13 +0000 (14:59 -0500)
commitac888986fc656c0badb0a4918b3581e2ebcb154a
tree0843e8d957e003637aa6f586c29e299ea9efcd4b
parentfcf70e0dbca1432959be5f3557acd546d639c9ba
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.1479325184@sss.pgh.pa.us>
src/bin/pg_dump/pg_backup_archiver.c