doc: minor update for description of "pg_roles" view

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

doc: minor update for description of "pg_roles" view

Ian Barwick-3
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

view-pg_roles.v1.patch (585 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

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

Bruce Momjian
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  <[hidden email]>        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 +


Reply | Threaded
Open this post in threaded view
|

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

Ian Barwick-3
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