pgsql: Lower minimum maintenance_work_mem to 64kB

From: John Naylor <john(dot)naylor(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Lower minimum maintenance_work_mem to 64kB
Date: 2024-08-10 08:19:03
Message-ID: E1schJX-003XoQ-8Q@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Lower minimum maintenance_work_mem to 64kB

Since the introduction of TID store, vacuum uses far less memory in
the common case than in versions 16 and earlier. Invoking multiple
rounds of index vacuuming in turn requires a much larger table. It'd
be a good idea anyway to cover this case in regression testing, and a
lower limit is less painful for slow buildfarm animals. The reason to
do it now is to re-enable coverage of the bugfix in commit 83c39a1f7f.

For consistency, give autovacuum_work_mem the same treatment.

Suggested by Andres Freund
Tested by Melanie Plageman
Backpatch to v17, where TID store was introduced

Discussion: https://postgr.es/m/20240516205458.ohvlzis5b5tvejru@awork3.anarazel.de
Discussion: https://postgr.es/m/20240722164745.fvaoh6g6zprisqgp%40awork3.anarazel.de

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/bbf668d66fbf61d45f1c187b08a5f51537bfb7c7

Modified Files
--------------
src/backend/postmaster/autovacuum.c | 6 +++---
src/backend/utils/misc/guc_tables.c | 7 ++++++-
src/backend/utils/misc/postgresql.conf.sample | 4 ++--
3 files changed, 11 insertions(+), 6 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message John Naylor 2024-08-10 08:19:12 pgsql: Lower minimum maintenance_work_mem to 64kB
Previous Message Peter Eisentraut 2024-08-10 06:41:44 pgsql: Fix inappropriate uses of atol()