granicus.if.org Git - postgresql/commit

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

author	Tom Lane <tgl@sss.pgh.pa.us>
	Fri, 25 Mar 2016 16:33:16 +0000 (12:33 -0400)
committer	Tom Lane <tgl@sss.pgh.pa.us>
	Fri, 25 Mar 2016 16:33:16 +0000 (12:33 -0400)
commit	c94959d4110a1965472956cfd631082a96f64a84
tree	603f7821778785ae423809a2a1d7ed4b0148d68b	tree \| snapshot
parent	d543170f2fdd6d9845aaf91dc0f6be7a2bf0d9e7	commit \| diff

Fix DROP OPERATOR to reset oprcom/oprnegate links to the dropped operator.

This avoids leaving dangling links in pg_operator; which while fairly
harmless are also unsightly.

While we're at it, simplify OperatorUpd, which went through
heap_modify_tuple for no very good reason considering it had already made
a tuple copy it could just scribble on.

Roma Sokolov, reviewed by Tomas Vondra, additional hacking by Robert Haas
and myself.

src/backend/catalog/pg_operator.c		diff \| blob \| history
src/backend/commands/operatorcmds.c		diff \| blob \| history
src/include/catalog/pg_operator_fn.h		diff \| blob \| history
src/test/regress/expected/drop_operator.out	[new file with mode: 0644]	blob
src/test/regress/parallel_schedule		diff \| blob \| history
src/test/regress/serial_schedule		diff \| blob \| history
src/test/regress/sql/drop_operator.sql	[new file with mode: 0644]	blob

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

RSS Atom