Re[2]: doc - improve description of default privileges

From: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
To: Bradley DeJong <bpd0018(at)gmail(dot)com>
Cc: PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re[2]: doc - improve description of default privileges
Date: 2018-08-30 07:33:41
Message-ID: alpine.DEB.2.21.1808300926250.14012@lancre
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Hello Bradley,

> comments on the patch ...

Thanks for the review.

> 1) I think that adding the "This privilege is abbreviated ... when
> displayed." lines to the privilege descriptions is redundant. The
> abbreviations are already listed after the "The entries shown by \dp are
> interpreted thus:" line. Just change that line to something like "The entries
> shown by the psql backslash commands, like \dp, are interpreted thus:".

Ok, removed.

> 2) I think that the psql command table should go with the current text on
> "Use psql's \dp command to obtain ..." rather than in the Examples section.
> It seems like changing the "For non-table objects there are other \d commands
> ..." line to an introductory comment like "The following table lists the \d
> commands that are used for non-table objects along with the default
> privileges granted to the object's owner and PUBLIC.

Ok, moved to the previous section.

> 3) The table title, "Default hardcoded access privileges per object's type",
> seems incomplete because it does not mention the psql commands part of the
> table.

Ok, added reference to psql backslash commands in title.

Find v3 attached.

--
Fabien.

Attachment Content-Type Size
doc-default-perms-3.patch text/x-diff 4.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Konstantin Knizhnik 2018-08-30 08:00:43 Bug in slot.c and are replication slots ever used at Window?
Previous Message Noah Misch 2018-08-30 07:06:09 Re: Facility for detecting insecure object naming