Re: is_superuser is not documented - Mailing list pgsql-hackers

From bt22kawamotok
Subject Re: is_superuser is not documented
Date
Msg-id 8838a06e69f2e30f38042bb9adc5755c@oss.nttdata.com
Whole thread Raw
In response to Re: is_superuser is not documented  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: is_superuser is not documented
List pgsql-hackers
> On the other hand, it seems pretty silly that it's GUC_REPORT if
> we want to consider it private.  I've not checked the git history,
> but I bet that flag was added later with no thought about context.
> 
> If we are going to document this then we should at least remove
> the GUC_NO_SHOW_ALL flag and rewrite the comment.  I wonder whether
> the GUC_NO_RESET_ALL flag is needed either --- seems like the
> PGC_INTERNAL context protects it sufficiently.

> I wonder why this one is marked USERSET where the other is not.
> You'd think both of them need similar special-casing about how
> to handle SET.

Thanks for your review.

I have created a patch in response to your suggestion.
I wasn't sure about USERSET, so I only created documentation for 
is_superuser.

Regards,
Kotaro Kawamoto.
Attachment

pgsql-hackers by date:

Previous
From: Shinya Kato
Date:
Subject: Re: [PATCH]Feature improvement for MERGE tab completion
Next
From: Hamid Akhtar
Date:
Subject: Re: Allow pageinspect's bt_page_stats function to return a set of rows instead of a single row