Re: allow granting CLUSTER, REFRESH MATERIALIZED VIEW, and REINDEX
Isaac Morland <isaac.morland@gmail.com>
From: Isaac Morland <isaac.morland@gmail.com>
To: Jeff Davis <pgsql@j-davis.com>
Cc: Pavel Luzanov <p.luzanov@postgrespro.ru>,
Nathan Bossart <nathandbossart@gmail.com>, pgsql-hackers@postgresql.org
Date: 2022-12-14T22:40:14Z
Lists: pgsql-hackers
On Wed, 14 Dec 2022 at 15:57, Jeff Davis <pgsql@j-davis.com> wrote: > On Wed, 2022-12-14 at 15:32 -0500, Isaac Morland wrote: > > > Is there a firm decision on the issue of changing the cluster index > > of a table? Re-clustering a table on the same index is clearly > > something that should be granted by MAINTAIN as I imagine it, but > > changing the cluster index, strictly speaking, changes the schema and > > could be considered outside of the scope of what should be allowed. > > On the other hand, I can see simplicity in having CLUSTER check the > > same permissions whether or not the cluster index is being updated. > > In both the case of CLUSTER and REFRESH, I don't have a strong enough > opinion to break them out into separate privileges. There's some > argument that can be made; but at the same time it's hard for me to > imagine someone really making use of the privileges separately. > Thanks, that makes a lot of sense. I wanted to make sure the question was considered. I'm very pleased this is happening and appreciate all the work you're doing. I have a few places where I want to be able to grant MAINTAIN so I'll be using this as soon as it's available on our production database.
Commits
-
Improve privilege documentation for maintenance commands.
- c2122aae636d 16.0 landed
-
Fix cache lookup hazards introduced by ff9618e82a.
- 4dbdb82513b6 16.0 landed
-
Move bool parameter for vacuum_rel() to option bits.
- 5b1a87994312 16.0 landed
-
Fix MAINTAIN privileges for toast tables and partitions.
- ff9618e82a46 16.0 landed
-
Add grantable MAINTAIN privilege and pg_maintain role.
- 60684dd834a2 16.0 landed