granicus.if.org Git - postgresql/commit

]> granicus.if.org Git - postgresql/commit

author	Alvaro Herrera <alvherre@alvh.no-ip.org>
	Thu, 23 Jan 2014 17:40:29 +0000 (14:40 -0300)
committer	Alvaro Herrera <alvherre@alvh.no-ip.org>
	Thu, 23 Jan 2014 17:40:29 +0000 (14:40 -0300)
commit	b152c6cd0de1827ba58756e24e18110cf902182a
tree	82d0ced551764509a60014e9996f3b4d634b4071	tree \| snapshot
parent	9f80f4835a55a1cbffcda5d23a617917f3286c14	commit \| diff

Make DROP IF EXISTS more consistently not fail

Some cases were still reporting errors and aborting, instead of a NOTICE
that the object was being skipped. This makes it more difficult to
cleanly handle pg_dump --clean, so change that to instead skip missing
objects properly.

Per bug #7873 reported by Dave Rolsky; apparently this affects a large
number of users.

Authors: Pavel Stehule and Dean Rasheed. Some tweaks by Álvaro Herrera

17 files changed:

src/backend/catalog/namespace.c		diff \| blob \| history
src/backend/catalog/objectaddress.c		diff \| blob \| history
src/backend/commands/dropcmds.c		diff \| blob \| history
src/backend/commands/functioncmds.c		diff \| blob \| history
src/backend/commands/opclasscmds.c		diff \| blob \| history
src/backend/commands/tablecmds.c		diff \| blob \| history
src/backend/commands/typecmds.c		diff \| blob \| history
src/backend/parser/parse_func.c		diff \| blob \| history
src/backend/parser/parse_oper.c		diff \| blob \| history
src/backend/parser/parse_type.c		diff \| blob \| history
src/backend/utils/adt/regproc.c		diff \| blob \| history
src/include/catalog/namespace.h		diff \| blob \| history
src/include/parser/parse_type.h		diff \| blob \| history
src/pl/plpgsql/src/pl_comp.c		diff \| blob \| history
src/test/regress/expected/drop_if_exists.out		diff \| blob \| history
src/test/regress/expected/event_trigger.out		diff \| blob \| history
src/test/regress/sql/drop_if_exists.sql		diff \| blob \| history

Unnamed repository; edit this file 'description' to name the repository.

RSS Atom