Re: [PoC] Improve dead tuple storage for lazy vacuum

From: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
To: John Naylor <john(dot)naylor(at)enterprisedb(dot)com>
Cc: Nathan Bossart <nathandbossart(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, Yura Sokolov <y(dot)sokolov(at)postgrespro(dot)ru>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PoC] Improve dead tuple storage for lazy vacuum
Date: 2023-03-10 15:26:18
Message-ID: CAD21AoDBmD5q=eO+K=gyuVt53XvwpJ2dgxPwrtZ-eVOjVmtJjg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Mar 10, 2023 at 11:30 PM Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
>
> On Fri, Mar 10, 2023 at 3:42 PM John Naylor
> <john(dot)naylor(at)enterprisedb(dot)com> wrote:
> >
> > On Thu, Mar 9, 2023 at 1:51 PM Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
> >
> > > I've attached the new version patches. I merged improvements and fixes
> > > I did in the v29 patch.
> >
> > I haven't yet had a chance to look at those closely, since I've had to devote time to other commitments. I remember I wasn't particularly impressed that v29-0008 mixed my requested name-casing changes with a bunch of other random things. Separating those out would be an obvious way to make it easier for me to look at, whenever I can get back to this. I need to look at the iteration changes as well, in addition to testing memory measurement (thanks for the new results, they look encouraging).
>
> Okay, I'll separate them again.

Attached new patch series. In addition to separate them again, I've
fixed a conflict with HEAD.

Regards,

--
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com

Attachment Content-Type Size
v31-0013-Add-min-and-max-classes-for-node3-and-node125.patch application/octet-stream 11.9 KB
v31-0011-Remove-the-max-memory-deduction-from-TidStore.patch application/octet-stream 3.9 KB
v31-0009-Review-vacuum-integration.patch application/octet-stream 13.2 KB
v31-0007-Review-radix-tree.patch application/octet-stream 22.5 KB
v31-0014-Revert-building-benchmark-module-for-CI.patch application/octet-stream 694 bytes
v31-0005-Tool-for-measuring-radix-tree-and-tidstore-perfo.patch application/octet-stream 24.7 KB
v31-0008-Review-TidStore.patch application/octet-stream 32.1 KB
v31-0003-Add-radixtree-template.patch application/octet-stream 117.0 KB
v31-0006-Use-TIDStore-for-storing-dead-tuple-TID-during-l.patch application/octet-stream 48.1 KB
v31-0002-Move-some-bitmap-logic-out-of-bitmapset.c.patch application/octet-stream 6.0 KB
v31-0012-Revert-the-update-for-the-minimum-value-of-maint.patch application/octet-stream 1.4 KB
v31-0001-Introduce-helper-SIMD-functions-for-small-byte-a.patch application/octet-stream 2.9 KB
v31-0010-Radix-tree-optionally-tracks-memory-usage-when-R.patch application/octet-stream 8.0 KB
v31-0004-Add-TIDStore-to-store-sets-of-TIDs-ItemPointerDa.patch application/octet-stream 36.0 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jeff Davis 2023-03-10 15:48:13 Re: pgsql: Use ICU by default at initdb time.
Previous Message Tom Lane 2023-03-10 15:09:26 Re: Date-Time dangling unit fix