Re: [PATCH] pg_permissions

From: "Joel Jacobson" <joel(at)compiler(dot)org>
To: "Chapman Flack" <chap(at)anastigmatix(dot)net>, pgsql-hackers(at)lists(dot)postgresql(dot)org, "Alvaro Herrera" <alvherre(at)alvh(dot)no-ip(dot)org>, "Peter Eisentraut" <peter(dot)eisentraut(at)enterprisedb(dot)com>
Subject: Re: [PATCH] pg_permissions
Date: 2024-06-12 22:14:09
Message-ID: 0c5a6b79-408c-4910-9b2e-4aa9a7b30f3a@app.fastmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Mar 10, 2022, at 22:02, Chapman Flack wrote:
> It looked to me as if the -hackers messages of 25 and 26 March 2021 had
> found a consensus that a pg_get_acl() function would be a good thing,
> with the views to be implemented over that.
>
> I'm just not seeing any later patch that adds such a function.

My apologies for late reply. Here it comes.

Recap: This patch is about adding two new system views: pg_privileges and pg_ownerships.

Changes since patch 0005 from 2021-03-25:

- Implement SQL-callable pg_get_acl()
This is a stripped down version of AlterObjectOwner_internal() from alter.c.

- Rename pg_permissions -> pg_privileges

- Use pg_shdepend + pg_get_acl() in pg_privileges, to avoid slow UNION ALL.

- Fix indentation of the new system views to be consistent with the other views.

- Add documentation of pg_get_acl() to func.sgml

- Move documentation of system views from catalogs.sgml to system-views.sgml

- Much smaller patch, thanks to getting rid of the long UNION ALL view definition:
1 file changed, 195 insertions(+), 460 deletions(-)

/Joel

Attachment Content-Type Size
0006-pg_privileges-and-pg_ownerships.patch application/octet-stream 17.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Melanie Plageman 2024-06-12 22:34:16 Re: RFC: adding pytest as a supported test framework
Previous Message Peter Geoghegan 2024-06-12 21:59:14 Re: Harmonizing pg_bsd_indent parameter names