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

From: Bertrand Drouvot <bertranddrouvot(dot)pg(at)gmail(dot)com>
To: Sami Imseih <samimseih(at)gmail(dot)com>
Cc: Alex Friedman <alexf01(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-21 09:43:43
Message-ID: Z7hKzz68VmLS8+N0@ip-10-97-1-34.eu-west-3.compute.internal
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On Tue, Feb 04, 2025 at 01:23:28PM -0600, Sami Imseih wrote:
> I confirmed the 20GB value as is described here [1].
> 8k page can hold 409 member groups and each
> member group can hold 4 members, thus
> (2^32/(409 *4))*8192 = 20GB.
>

Thanks for the patch!

I do agree that that's a bug in the doc.

Indeed, while I do agree that MULTIXACT_MEMBER_SAFE_THRESHOLD can be seen
as "2G" (not 2GB...) then it's compared in MultiXactMemberFreezeThreshold() to
"members" which is the number of members: So, the member's size has to be taken
into account.

Maybe we could also add a comment in multixact.c to update the doc accordingly if
the computation changes? (I think that will be easy to miss).

Regards,

--
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Zharkov Roman 2025-02-21 09:44:37 Re: TAP test started using meson, can get a tcp port already used by another test.
Previous Message Hayato Kuroda (Fujitsu) 2025-02-21 09:36:35 RE: Adding a '--clean-publisher-objects' option to 'pg_createsubscriber' utility.