[COMMITTERS] pgsql: Improve memory management in autovacuum.c. - Mailing list pgsql-committers

From Tom Lane
Subject [COMMITTERS] pgsql: Improve memory management in autovacuum.c.
Date
Msg-id E1dvoDy-0006aY-Ru@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Improve memory management in autovacuum.c.

Invoke vacuum(), as well as "work item" processing, in the PortalContext
that do_autovacuum() has manufactured, which will be reset before each
such invocation.  This ensures cleanup of any memory leaked by these
operations.  It also avoids the rather dangerous practice of calling
vacuum() in a context that vacuum() itself will destroy while it runs.
There's no known live bug there, but it's not hard to imagine introducing
one if we leave it like this.

Tom Lane, reviewed by Michael Paquier and Alvaro Herrera

Discussion: https://postgr.es/m/13849.1506114543@sss.pgh.pa.us

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/335f3d04e4c8dd495c4dd30ab1049b6fe8f25052

Modified Files
--------------
src/backend/postmaster/autovacuum.c | 22 +++++++++++++++++-----
1 file changed, 17 insertions(+), 5 deletions(-)


--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: [COMMITTERS] pgsql: Ten-second timeout in 013_crash_restart.pl is not enough,let's
Next
From: Fabien COELHO
Date:
Subject: Re: [COMMITTERS] pgsql: Remove pgbench "progress" test pendingsolution of its timing is