> 16 апр. 2020 г., в 17:46, Magnus Hagander <mag...@hagander.net> написал(а):
>
>
> If we do that, it may be better that we define "PGSTAT_VIEW_PRIV()" or
> something like and replace the all occurances of the idiomatic
> condition with it.
>
> You mean something like the attached?
>
> <allow_read_all_stats3.diff>
Is it correct that we use DEFAULT_ROLE_READ_ALL_STATS regardless of
inheritance? I'm not familiar with what is inherited and what is not, so I
think it's better to ask explicitly.
+#define HAS_PGSTAT_PERMISSIONS(role) (is_member_of_role(GetUserId(),
DEFAULT_ROLE_READ_ALL_STATS) || has_privs_of_role(GetUserId(), role))
Besides this, the patch looks good to me.
Thanks!
Best regards, Andrey Borodin,