|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: doc - improve description of default privileges|
|Views:||Raw Message | Whole Thread | Download mbox|
> 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:".
> 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
Ok, added reference to psql backslash commands in title.
Find v3 attached.
|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|