Re: Additional role attributes && superuser review - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Additional role attributes && superuser review
Date
Msg-id CAB7nPqTeYLt5hXKYx+u3f84-xh+M0k0FOMpvM_CgbHgaap6m4w@mail.gmail.com
Whole thread Raw
In response to Re: Additional role attributes && superuser review  (Stephen Frost <sfrost@snowman.net>)
Responses Re: Additional role attributes && superuser review  (Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>)
List pgsql-hackers
On Tue, Dec 22, 2015 at 1:41 AM, Stephen Frost <sfrost@snowman.net> wrote:
> Updated and rebased patch attached which takes the 'pg_switch_xlog'
> default role back out, leaving us with:
>
> pg_monitor - View privileged info
> pg_backup - start/stop backups, switch xlog, create restore points
> pg_replay - Pause/resume xlog replay on replicas
> pg_replication - Create/destroy/etc replication slots
> pg_rotate_logfile - Request logfile rotation
> pg_signal_backend - Signal other backends (cancel query/terminate)
> pg_file_settings - View configuration settings in all config files
>
> Michael, another review would be great, if you don't mind.  I'm going to
> be going through it also more closely since it sounds like we've got
> consensus on at least this initial set of default roles and rights.  If
> all looks good then I'll commit it.

Thanks. This looks fine to me. I just have one single comment:

+       <entry>Request logfile rotation.</entry>
s/logfile/transaction log file/
-- 
Michael



pgsql-hackers by date:

Previous
From: Kenan Yao
Date:
Subject: Re: A question regarding LWLock in ProcSleep
Next
From: Michael Paquier
Date:
Subject: Re: custom function for converting human readable sizes to bytes