From: | Simon Riggs <simon(at)2ndquadrant(dot)com> |
---|---|
To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
Cc: | Feike Steenbergen <feikesteenbergen(at)gmail(dot)com>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Fix permissions check on pg_stat_get_wal_senders |
Date: | 2018-01-05 14:19:46 |
Message-ID: | CANP8+j+s-wAOsasRygk46mAM+Pxu-nHUTTqdv57C6=bjWVwqDQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 23 December 2017 at 10:56, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> wrote:
> On Fri, Dec 22, 2017 at 07:49:34AM +0100, Feike Steenbergen wrote:
>> On 21 December 2017 at 14:11, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> wrote:
>> > You mean a WAL receiver here, not a WAL sender.
>>
>> Fixed, thanks
>
> [nit]
> /*
> - * Only superusers can see details. Other users only get the pid value
> + * Only superusers and members of pg_read_all_stats can see details.
> + * Other users only get the pid value
> * to know whether it is a WAL receiver, but no details.
> */
>
> Incorrect comment format.
> [/nit]
>
> Committers run pgindent on each patch before committing anyway, and what
> you are proposing here looks good to me, so I am marking that as ready for
> committer. Simon, as the original committer of 25fff407, could you look
> at what is proposed here?
Yup, I got this.
--
Simon Riggs http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2018-01-05 14:27:18 | Re: [HACKERS] Creating backup history files for backups taken from standbys |
Previous Message | Simon Riggs | 2018-01-05 14:13:45 | Re: [HACKERS] Timeline ID in backup_label file |