Re: Adding REPACK [concurrently]

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Antonin Houska <ah(at)cybertec(dot)at>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, Mihail Nikalayeu <mihailnikalayeu(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Srinath Reddy Sadipiralla <srinath2133(at)gmail(dot)com>, Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Robert Treat <rob(at)xzilla(dot)net>
Subject: Re: Adding REPACK [concurrently]
Date: 2026-04-20 16:30:25
Message-ID: aeZG8QZKMDGdcnK_@alvherre.pgsql
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2026-Apr-20, Alvaro Herrera wrote:

> On 2026-Apr-20, Tom Lane wrote:

> > I think you need to move these tests out into some other test suite
> > (or make a new one).
>
> I'll see what I can find.

I think the simplest would be to add them to
src/test/modules/injection_points. We already have some repack tests
there (because they needed injection points), and it has an SQL suite,
so we would not be adding any extra overhead. However, the new tests
are not related to injection points, so they would be out of place.

Another possibility could be src/test/subscription, but it doesn't have
sql tests; doesn't seem good to have them just for this.

There's also contrib/test_decoding. It's somewhat vaguely adjacent, and
the tests aren't _really_ about the decoding part, but of all these
options, it seems the least bad one.

I don't like the idea of adding another suite. Too much scaffolding for
so little, I think.

I don't have any other ideas ATM.

--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/
"We're here to devour each other alive" (Hobbes)

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Fujii Masao 2026-04-20 16:35:34 Re: xact_rollback spikes when logical walsender exits
Previous Message Melanie Plageman 2026-04-20 16:18:24 Re: eliminate xl_heap_visible to reduce WAL (and eventually set VM on-access)