MIME-Version: 1.0
References: 
 <CADzfLwWFr9h_+cbSQvPpdxgLbVL5wwxFRx21ezNvLYgJM=FVCQ@mail.gmail.com>
 <202604062213.cgo352cdsgsm@alvherre.pgsql>
 <OS9PR01MB121498680B79ECB53367E269EF55AA@OS9PR01MB12149.jpnprd01.prod.outlook.com>
 <225003.1775571560@localhost>
 <CAA4eK1KWDbBk4FgbbWdivQLrPPzR4zgvfnHK4WjWC78rbuRVbg@mail.gmail.com>
In-Reply-To: 
 <CAA4eK1KWDbBk4FgbbWdivQLrPPzR4zgvfnHK4WjWC78rbuRVbg@mail.gmail.com>
From: Masahiko Sawada <sawada.mshk@gmail.com>
Date: Fri, 8 May 2026 16:22:04 -0700
Message-ID: 
 <CAD21AoAAyERNCFaG20ExXNquWkv3GELJ3CuYQnKc13uhSsnfYA@mail.gmail.com>
Subject: Re: Adding REPACK [concurrently]
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: Antonin Houska <ah@cybertec.at>,
 "Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>,
	Alvaro Herrera <alvherre@alvh.no-ip.org>,
 Mihail Nikalayeu <mihailnikalayeu@gmail.com>,
	Srinath Reddy Sadipiralla <srinath2133@gmail.com>,
	Matthias van de Meent <boekewurm+postgres@gmail.com>,
	Pg Hackers <pgsql-hackers@lists.postgresql.org>,
 Robert Treat <rob@xzilla.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CAD21AoAAyERNCFaG20ExXNquWkv3GELJ3CuYQnKc13uhSsnfYA%40mail.gmail.com>
Precedence: bulk

On Tue, Apr 7, 2026 at 8:49=E2=80=AFPM Amit Kapila <amit.kapila16@gmail.com=
> wrote:
>
> On Tue, Apr 7, 2026 at 7:49=E2=80=AFPM Antonin Houska <ah@cybertec.at> wr=
ote:
> >
> > Hayato Kuroda (Fujitsu) <kuroda.hayato@fujitsu.com> wrote:
> >
> > > 02. SnapBuildProcessRunningXacts
> > >
> > > Per my understanding, the db_specic snapshot can be also serialized. =
Is it
> > > possibility tha normal logical decoding system restores the snapshot =
and obtain
> > > the wrong result?
> >
> > I don't think that the database-specific xl_running_xacts WAL record af=
fects
> > what SnapBuildSerialize() writes to disk: the contents of builder->comm=
itted,
> > etc. is updated by decoding COMMIT and ABORT records.
> >
>
> I think the point is that the other process say a walsender could
> restore such a snapshot making it take the wrong decision.
>

Right. I think it affects even concurrent REPACK (CONCURRENTLY)
running on other databases. They could end up restoring the snapshot
serialized by another REPACK command running on another database and
becoming the consistent state without waiting for transactions
concurrently running on the same database, which is clearly wrong.

Regards,

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