pgsql: Fix erroneous handling of shared dependencies (ie dependencies on - Mailing list pgsql-committers

From tgl@postgresql.org (Tom Lane)
Subject pgsql: Fix erroneous handling of shared dependencies (ie dependencies on
Date
Msg-id 20091002181332.8FB3D753FB7@cvs.postgresql.org
Whole thread Raw
List pgsql-committers
Log Message:
-----------
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.

Tags:
----
REL8_1_STABLE

Modified Files:
--------------
    pgsql/doc/src/sgml/ref:
        create_function.sgml (r1.67.2.2 -> r1.67.2.3)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/create_function.sgml?r1=1.67.2.2&r2=1.67.2.3)
        create_view.sgml (r1.31 -> r1.31.2.1)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/create_view.sgml?r1=1.31&r2=1.31.2.1)
    pgsql/src/backend/catalog:
        pg_proc.c (r1.135.2.1 -> r1.135.2.2)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/catalog/pg_proc.c?r1=1.135.2.1&r2=1.135.2.2)

pgsql-committers by date:

Previous
From: tgl@postgresql.org (Tom Lane)
Date:
Subject: pgsql: Fix erroneous handling of shared dependencies (ie dependencies on
Next
From: tgl@postgresql.org (Tom Lane)
Date:
Subject: pgsql: Make sure that GIN fast-insert and regular code paths enforce the