Re: Additional role attributes && superuser review

Поиск
Список
Период
Сортировка
От Adam Brightwell
Тема Re: Additional role attributes && superuser review
Дата
Msg-id CAKRt6CQJY-jhkyggB5nCTC8wf3Md_kH_VmLdQbK0pWL5zX=2pw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Additional role attributes && superuser review  (Robert Haas <robertmhaas@gmail.com>)
Ответы Re: Additional role attributes && superuser review  (Stephen Frost <sfrost@snowman.net>)
Список pgsql-hackers


On Mon, Jan 5, 2015 at 11:49 AM, Robert Haas <robertmhaas@gmail.com> wrote:
On Wed, Dec 24, 2014 at 12:48 PM, Adam Brightwell
<adam.brightwell@crunchydatasolutions.com> wrote:
> * BACKUP - allows role to perform backup operations
> * LOGROTATE - allows role to rotate log files
> * MONITOR - allows role to view pg_stat_* details
> * PROCSIGNAL - allows role to signal backend processes

How about just "SIGNAL" instead of "PROCSIGNAL"?

Sure. 
 
Generally, I think we'll be happier if these capabilities have names
that are actual words - or combinations of words - rather than partial
words, so I'd suggest avoiding things like PROC for PROCESS and AUTH
for AUTHORIZATION.

Agreed.
 
In this particular case, it seems like the name of the capability is
based off the name of an internal system data structure.  That's the
sort of thing that we do not want to expose to users.  As far as we
can, we should try to describe what the capability allows, not the
details of how that is (currently) implemented.

Agreed.

If others are also in agreement on this point then I'll update the patch accordingly.

Thanks,
Adam 

--

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Dan Langille
Дата:
Сообщение: PGCon 2015 call for papers
Следующее
От: Heikki Linnakangas
Дата:
Сообщение: Re: pg_rewind in contrib