Re: Adding REPACK [concurrently]

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: Antonin Houska <ah(at)cybertec(dot)at>
Cc: Srinath Reddy Sadipiralla <srinath2133(at)gmail(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Mihail Nikalayeu <mihailnikalayeu(at)gmail(dot)com>, Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, Pg Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Robert Treat <rob(at)xzilla(dot)net>
Subject: Re: Adding REPACK [concurrently]
Date: 2026-04-05 20:41:50
Message-ID: 202604052035.il2hp2wze6pz@alvherre.pgsql
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

So here's a v55 version of the base REPACK patches that I'm feeling
comfortable calling very close to committable. I'm going to give an
additional read tomorrow and maybe make cosmetic adjustments, but there
should be nothing substantial. Of course, the subsequent additions in
the other patches of v54 are still in the cards, and they are most
likely essential.

Changes compared to v54:
- changed reform_tuple() to not deform the tuple if no attributes are
going to be touched. We can simply make a copy instead, which I
suspect is considerably cheaper (but I didn't measure).

- cleaned up worker shmem shutdown callback. I think it's how it is
because it copied parallel worker code, but that has a weird structure
for --as far as I can see-- no good reason (we oughta change it too)

- renamed the worker from pgoutput_repack to pgrepack.
(Note that this is an internal name that users don't face.)

- reverted some unnecessary changes to master

Thanks

--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/

Attachment Content-Type Size
v55-0001-Rename-cluster.c-h-repack.c-h.patch text/x-diff 7.8 KB
v55-0002-Add-CONCURRENTLY-option-to-REPACK-command.patch text/x-diff 173.1 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2026-04-05 21:02:28 Re: Stack-based tracking of per-node WAL/buffer usage
Previous Message Tom Lane 2026-04-05 20:15:03 Re: CREATE SCHEMA ... CREATE DOMAIN support