Re: Allow pg_signal_backend members to use pg_log_backend_memory_stats().

From: "Bossart, Nathan" <bossartn(at)amazon(dot)com>
To: Jeff Davis <pgsql(at)j-davis(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Allow pg_signal_backend members to use pg_log_backend_memory_stats().
Date: 2021-10-25 21:26:47
Message-ID: A2C5306D-81AB-4309-9C9D-F7D02C992BCF@amazon.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 10/25/21, 1:43 PM, "Jeff Davis" <pgsql(at)j-davis(dot)com> wrote:
> On Mon, 2021-10-25 at 16:10 +0900, Michael Paquier wrote:
>> Hmm. Why don't you split the patch into two parts that can be
>> discussed separately then? There would be one to remove all the
>> superuser() checks you can think of, and a potential second to grant
>> those function's execution to some system role.
>
> Good idea. Attached a patch to remove the superuser check on
> pg_log_backend_memory_contexts(), except in the case when trying to log
> memory contexts of a superuser backend.

LGTM.

Nathan

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2021-10-25 21:30:38 Re: Allow pg_signal_backend members to use pg_log_backend_memory_stats().
Previous Message Mark Dilger 2021-10-25 21:12:16 Re: Predefined role pg_maintenance for VACUUM, ANALYZE, CHECKPOINT.