| 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: | Whole Thread | Raw Message | 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 |
| 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 |