From: | Reid Thompson <reid(dot)thompson(at)crunchydata(dot)com> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | reid(dot)thompson(at)crunchydata(dot)com, pgsql-hackers(at)lists(dot)postgresql(dot)org, "Drouvot, Bertrand" <bertranddrouvot(dot)pg(at)gmail(dot)com>, Melanie Plageman <melanieplageman(at)gmail(dot)com> |
Subject: | Re: Add tracking of backend memory allocated to pg_stat_activity |
Date: | 2022-12-06 13:47:55 |
Message-ID: | cbe20863f2d24ddede079042291f672123446f16.camel@crunchydata.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, 2022-12-02 at 09:18 -0800, Andres Freund wrote:
> Hi,
>
> On 2022-12-02 11:09:30 -0500, Reid Thompson wrote:
> > It appears to me that Postmaster populates the local variable that
> > *my_allocated_bytes points to. That allocation is passed to forked
> > children, and if not zeroed out, will be double counted as part of
> > the child allocation. Is this invalid?
>
> I don't think we should count allocations made before
> backend_status.c has
> been initialized.
>
> Greetings,
>
> Andres Freund
Hi,
The intent was to capture and display all the memory allocated to the
backends, for admins/users/max_total_backend_memory/others to utilize.
Why should we ignore the allocations prior to backend_status.c?
Thanks,
Reid
--
Reid Thompson
Senior Software Engineer
Crunchy Data, Inc.
reid(dot)thompson(at)crunchydata(dot)com
www.crunchydata.com
From | Date | Subject | |
---|---|---|---|
Next Message | Masahiko Sawada | 2022-12-06 13:48:18 | Re: Force streaming every change in logical decoding |
Previous Message | Pavel Borisov | 2022-12-06 13:44:50 | Re: [PATCH] Check snapshot argument of index_beginscan and family |