Re: [PATCH] Add ACL (Access Control List) acronym - Mailing list pgsql-hackers

From David G. Johnston
Subject Re: [PATCH] Add ACL (Access Control List) acronym
Date
Msg-id CAKFQuwYhPDPVCQ2AFWdgoNBqWWWEvgsww3C251vtbMmRZAXp+A@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Add ACL (Access Control List) acronym  (Nathan Bossart <nathandbossart@gmail.com>)
Responses Re: [PATCH] Add ACL (Access Control List) acronym
List pgsql-hackers
On Mon, Jun 24, 2024 at 8:44 AM Nathan Bossart <nathandbossart@gmail.com> wrote:
On Mon, Jun 24, 2024 at 02:32:27PM +0200, Joel Jacobson wrote:
> This patch is based on a suggestion from a separate thread [1]:
>
> On Mon, Jun 24, 2024, at 01:46, Michael Paquier wrote:
>> Rather unrelated to this patch, still this patch makes the situation
>> more complicated in the docs, but wouldn't it be better to add ACL as
>> a term in acronyms.sql, and reuse it here?  It would be a doc-only
>> patch that applies on top of the rest (could be on a new thread of its
>> own), with some <acronym> markups added where needed.

Sounds reasonable to me.

+1


+      <ulink url="https://en.wikipedia.org/wiki/Access_Control_List">Access Control List, i.e. privileges list</ulink>

I think we could omit "i.e. privileges list."


Agreed.  Between the docs and code we say "privileges list" once and that refers to the dumputIls description of the arguments to grant.  As the acronym page now defines the term using fundamentals, introducing another term not used elsewhere seems undesirable.

Observations:
We are referencing a disambiguation page.  We never actually spell out ACL anywhere so we might as well just reference what Wikipedia believes is the expected spelling.

The page we link to uses "permissions" while we consistently use "privileges" to describe the contents of the list.  This seems like an obvious synonym, but as the point of these is to formally define things, pointing this equivalence is worth considering.

David J.

pgsql-hackers by date:

Previous
From: Matthias van de Meent
Date:
Subject: Re: Improve EXPLAIN output for multicolumn B-Tree Index
Next
From: Peter Geoghegan
Date:
Subject: Re: POC, WIP: OR-clause support for indexes