| From: | Andres Freund <andres(at)anarazel(dot)de> |
|---|---|
| To: | Peter Geoghegan <pg(at)bowt(dot)ie> |
| Cc: | Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, Jeff Davis <pgsql(at)j-davis(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: pg_walinspect memory leaks |
| Date: | 2023-02-14 00:55:37 |
| Message-ID: | 20230214005537.57zljs3hgmto7ajw@awork3.anarazel.de |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On 2023-02-13 15:22:02 -0800, Peter Geoghegan wrote:
> More concretely, it looks like GetWALRecordInfo() calls
> CStringGetTextDatum/cstring_to_text in a way that accumulates way too
> much memory in ExprContext.
Additionally, we leak two stringinfos for each record.
> This could be avoided by using a separate memory context that is reset
> periodically, or something else along the same lines.
Everything other than a per-row memory context that's reset each time seems
hard to manage in this case.
Somehwat funnily, GetWALRecordsInfo() then ends up being unnecessarily
dilligent about cleaning up O(1) memory, after not caring about O(N) memory...
Greetings,
Andres Freund
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Nathan Bossart | 2023-02-14 00:55:58 | Re: recovery modules |
| Previous Message | Andres Freund | 2023-02-14 00:43:10 | Re: Adding "large" to PG_TEST_EXTRA |