Re: Doc fix of aggressive vacuum threshold for multixact members storage

From: Alex Friedman <alexf01(at)gmail(dot)com>
To: John Naylor <johncnaylorls(at)gmail(dot)com>
Cc: Sami Imseih <samimseih(at)gmail(dot)com>, Bertrand Drouvot <bertranddrouvot(dot)pg(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Doc fix of aggressive vacuum threshold for multixact members storage
Date: 2025-02-26 16:50:40
Message-ID: 695cfd63-f766-4530-9b2a-34c1eae21ab7@gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi John,

Thanks for reviewing.

> It seems at a minimum this one-line patch is sufficient for the correction:
>
> - storage occupied by multixacts members exceeds 2GB, aggressive vacuum
> + storage occupied by multixacts members exceeds about 10GB,
> aggressive vacuum
>
> Commit c552e171d16e removed the percentage as part of a judgment call
> on clarity, and I'm not sure that was wrong.
>
> We could add the proposed language on "can grow up to about 20GB" at
> the end of this paragraph, which seems more natural -- first mention
> the amount that triggers aggressive vacuum, then the maximum size.

Yes, I believe this can work.

> I'm on the fence about putting a hint in the C file, but the
> computation has changed in the past, see commit b4d4ce1d50bbdf , so
> it's a reasonable idea.

That's a good find about the change. Taken together with Bertrand's comments, I've added two reminders to multixact.c to update the docs, one for the threshold and another for the multixact storage scheme. Please see if it makes sense.

v4 patch attached.

Best regards,

Alex Friedman

Attachment Content-Type Size
v4-0001-Doc-fix-of-aggressive-vacuum-threshold-for-multix.patch text/plain 2.6 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Bertrand Drouvot 2025-02-26 17:13:19 Re: Draft for basic NUMA observability
Previous Message Corey Huinker 2025-02-26 16:43:03 Re: Statistics Import and Export