doc: minor update for description of "pg_roles" view

Started by Ian Lawrence Barwickalmost 7 years ago3 messageshackers
Jump to latest
#1Ian Lawrence Barwick
barwick@gmail.com

Hi

Here:

https://www.postgresql.org/docs/12/view-pg-roles.html

we state:

"This view explicitly exposes the OID column of the underlying table,
since that is needed to do joins to other catalogs."

I think it's superfluous to mention this now OIDs are exposed by default;
attached patch (for REL_12_STABLE and HEAD) simply removes this sentence.

Regards

Ian Barwick

--
Ian Barwick https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

Attachments:

view-pg_roles.v1.patchtext/x-patch; name=view-pg_roles.v1.patchDownload+0-5
#2Bruce Momjian
bruce@momjian.us
In reply to: Ian Lawrence Barwick (#1)
Re: doc: minor update for description of "pg_roles" view

On Wed, Jul 10, 2019 at 02:35:56PM +0900, Ian Barwick wrote:

Hi

Here:

https://www.postgresql.org/docs/12/view-pg-roles.html

we state:

"This view explicitly exposes the OID column of the underlying table,
since that is needed to do joins to other catalogs."

I think it's superfluous to mention this now OIDs are exposed by default;
attached patch (for REL_12_STABLE and HEAD) simply removes this sentence.

Patch applied though PG 12. Thanks.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ As you are, so once was I.  As I am, so you will be. +
+                      Ancient Roman grave inscription +
#3Ian Lawrence Barwick
barwick@gmail.com
In reply to: Bruce Momjian (#2)
Re: doc: minor update for description of "pg_roles" view

On 7/11/19 3:24 AM, Bruce Momjian wrote:

On Wed, Jul 10, 2019 at 02:35:56PM +0900, Ian Barwick wrote:

Hi

Here:

https://www.postgresql.org/docs/12/view-pg-roles.html

we state:

"This view explicitly exposes the OID column of the underlying table,
since that is needed to do joins to other catalogs."

I think it's superfluous to mention this now OIDs are exposed by default;
attached patch (for REL_12_STABLE and HEAD) simply removes this sentence.

Patch applied though PG 12. Thanks.

Thanks!

Regards

Ian Barwick

--
Ian Barwick https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services