Thread: doc: minor update for description of "pg_roles" view

doc: minor update for description of "pg_roles" view

From
Ian Barwick
Date:
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

Attachment

Re: doc: minor update for description of "pg_roles" view

From
Bruce Momjian
Date:
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 +



Re: doc: minor update for description of "pg_roles" view

From
Ian Barwick
Date:
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