Re: [PATCH v2] use has_privs_for_role for predefined roles - Mailing list pgsql-hackers

From Bossart, Nathan
Subject Re: [PATCH v2] use has_privs_for_role for predefined roles
Date
Msg-id 5F9388A2-35EA-4DAA-BE05-68D55EEA6DC3@amazon.com
Whole thread Raw
In response to Re: [PATCH v2] use has_privs_for_role for predefined roles  (Joshua Brindle <joshua.brindle@crunchydata.com>)
Responses Re: [PATCH v2] use has_privs_for_role for predefined roles
List pgsql-hackers
On 11/8/21, 2:19 PM, "Joshua Brindle" <joshua.brindle@crunchydata.com> wrote:
> Thanks for the review, attached is an update with that comment fixed
> and also sgml documentation changes that I missed earlier.

I think there are a number of documentation changes that are still
missing.  I did a quick scan and saw the "is member of" language in
func.sgml, monitoring.sgml, pgbuffercache.sgml, pgfreespacemap.sgml,
pgrowlocks.sgml, pgstatstatements.sgml, and pgvisibility.sgml.

<para>
 By default, the <structname>pg_shmem_allocations</structname> view can be
-   read only by superusers or members of the <literal>pg_read_all_stats</literal>
-   role.
+   read only by superusers or roles with privilges of the
+   <literal>pg_read_all_stats</literal> role.
</para>
</sect1>

nitpick: "privileges" is misspelled.

Nathan


pgsql-hackers by date:

Previous
From: Bharath Rupireddy
Date:
Subject: Re: wait event and archive_command
Next
From: Bharath Rupireddy
Date:
Subject: Re: emit recovery stats via a new file or a new hook