Re: enhance pg_log_backend_memory_contexts() to log memory contexts of auxiliary processes

On Sun, Nov 28, 2021 at 7:15 PM Bharath Rupireddy
<bharath(dot)rupireddyforpostgres(at)gmail(dot)com> wrote:
>
> On Sun, Nov 28, 2021 at 5:21 PM vignesh C <vignesh21(at)gmail(dot)com> wrote:
> > Thanks for the updated patch, one comment:
> > 1) The function can be indented similar to other functions in the same file:
> > +CREATE FUNCTION memcxt_get_proc_pid(text)
> > +RETURNS int
> > +LANGUAGE SQL
> > +AS 'SELECT pid FROM pg_stat_activity WHERE backend_type = $1';
> >
> > Something like:
> > +CREATE FUNCTION memcxt_get_proc_pid(text)
> > + RETURNS int
> > + LANGUAGE SQL
> > + AS 'SELECT pid FROM pg_stat_activity WHERE backend_type = $1';
>
> Done. PSA v6 patch.

Thanks for the updated patch. The patch applies neatly, make
check-world passes and the documentation looks good. I did not find
any issues with the v6 patch, I'm marking the patch as Ready for
Committer.

Regards,
Vignesh