pgsql: Handle dependencies properly in ALTER POLICY - Mailing list pgsql-committers

From Stephen Frost
Subject pgsql: Handle dependencies properly in ALTER POLICY
Date
Msg-id E1a7UY9-0004kV-SJ@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Handle dependencies properly in ALTER POLICY

ALTER POLICY hadn't fully considered partial policy alternation
(eg: change just the roles on the policy, or just change one of
the expressions) when rebuilding the dependencies.  Instead, it
would happily remove all dependencies which existed for the
policy and then only recreate the dependencies for the objects
referred to in the specific ALTER POLICY command.

Correct that by extracting and building the dependencies for all
objects referenced by the policy, regardless of if they were
provided as part of the ALTER POLICY command or were already in
place as part of the pre-existing policy.

Branch
------
REL9_5_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/12a54c888cf7bd9c37c4ce420e84cb52debe0184

Modified Files
--------------
src/backend/commands/policy.c             |   97 +++++++++++++++++++++++++++++
src/test/regress/expected/rowsecurity.out |   43 +++++++++++++
src/test/regress/sql/rowsecurity.sql      |   31 +++++++++
3 files changed, 171 insertions(+)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Still more fixes for planner's handling of LATERAL references.
Next
From: Stephen Frost
Date:
Subject: pgsql: Handle dependencies properly in ALTER POLICY