From: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com> |
---|---|
To: | Andrey Borodin <x4mmm(at)yandex-team(dot)ru> |
Cc: | Gilles Darold <gilles(at)darold(dot)net>, Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, Alexander Korotkov <aekorotkov(at)gmail(dot)com>, Anastasia Lubennikova <a(dot)lubennikova(at)postgrespro(dot)ru>, Daniel Gustafsson <daniel(at)yesql(dot)se>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: MultiXact\SLRU buffers configuration |
Date: | 2021-03-24 21:31:56 |
Message-ID: | CA+hUKGJ4TwipnMDmKdORu4zUo_rofsuu5FQsgV=UHJ3bY38ERw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi Andrey,
On Sat, Mar 13, 2021 at 1:44 AM Andrey Borodin <x4mmm(at)yandex-team(dot)ru> wrote:
> [v10]
+int multixact_offsets_slru_buffers = 8;
+int multixact_members_slru_buffers = 16;
+int subtrans_slru_buffers = 32;
+int notify_slru_buffers = 8;
+int serial_slru_buffers = 16;
+int clog_slru_buffers = 0;
+int commit_ts_slru_buffers = 0;
I don't think we should put "slru" (the name of the buffer replacement
algorithm, implementation detail) in the GUC names.
+ It defaults to 0, in this case CLOG size is taken as
<varname>shared_buffers</varname> / 512.
We already know that increasing the number of CLOG buffers above the
current number hurts as the linear search begins to dominate
(according to the commit message for 5364b357), and it doesn't seem
great to ship a new feature that melts your CPU when you turn it up.
Perhaps, to ship this, we need to introduce a buffer mapping table? I
have attached a "one coffee" attempt at that, on top of your v10 patch
(unmodified), for discussion. It survives basic testing but I don't
know how it performs.
Attachment | Content-Type | Size |
---|---|---|
v10-0001-Make-all-SLRU-buffer-sizes-configurable.patch | text/x-patch | 19.5 KB |
v10-0002-Add-buffer-mapping-table-for-SLRUs.patch | text/x-patch | 7.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2021-03-24 22:44:02 | Re: pgsql: Move tablespace path re-creation from the makefiles to pg_regres |
Previous Message | Robert Haas | 2021-03-24 20:46:45 | Re: pg_amcheck contrib application |