Fix erroneous handling of shared dependencies (ie dependencies on roles)
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 2 Oct 2009 18:13:32 +0000 (18:13 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 2 Oct 2009 18:13:32 +0000 (18:13 +0000)
commit595f2744b0498e5b82649242aae35553761f3081
tree3a95213fd6fb73b94be6b958b1a1238081bc0781
parentfa990959f453a84e3aab11b24b10e0eea75471bc
Fix erroneous handling of shared dependencies (ie dependencies on roles)
in CREATE OR REPLACE FUNCTION.  The original code would update pg_shdepend
as if a new function was being created, even if it wasn't, with two bad
consequences: pg_shdepend might record the wrong owner for the function,
and any dependencies for roles mentioned in the function's ACL would be lost.
The fix is very easy: just don't touch pg_shdepend at all when doing a
function replacement.

Also update the CREATE FUNCTION reference page, which never explained
exactly what changes and doesn't change in a function replacement.
In passing, fix the CREATE VIEW reference page similarly; there's no
code bug there, but the docs didn't say what happens.
doc/src/sgml/ref/create_function.sgml
doc/src/sgml/ref/create_view.sgml
src/backend/catalog/pg_proc.c