Mime-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset=UTF-8
Date: Mon, 20 Oct 2025 15:19:50 -0300
Message-Id: <DDNCV8U74QPS.KBHN82IW5S4X@gmail.com>
From: "Matheus Alcantara" <matheusssilv97@gmail.com>
To: =?utf-8?q?=C3=81lvaro_Herrera?= <alvherre@kurilemu.de>, "Matheus
 Alcantara" <matheusssilv97@gmail.com>
Cc: "Joel Jacobson" <joel@compiler.org>, "Arseniy Mukhin"
 <arseniy.mukhin.dev@gmail.com>, "Masahiko Sawada" <sawada.mshk@gmail.com>,
 "Rishu Bagga" <rishu.postgres@gmail.com>, "Yura Sokolov"
 <y.sokolov@postgrespro.ru>, "Daniil Davydov" <3danissimo@gmail.com>,
 "Alexandra Wang" <alexandra.wang.oss@gmail.com>, "pgsql-hackers"
 <pgsql-hackers@postgresql.org>, "Tom Lane" <tgl@sss.pgh.pa.us>
Subject: Re: LISTEN/NOTIFY bug: VACUUM sets frozenxid past a xid in async
 queue
References: <DDN5OD2RQFF4.CW8X8CMVQAJC@gmail.com>
 <202510201414.56f2msqusjla@alvherre.pgsql>
In-Reply-To: <202510201414.56f2msqusjla@alvherre.pgsql>
Archived-At: 
 <https://www.postgresql.org/message-id/DDNCV8U74QPS.KBHN82IW5S4X%40gmail.com>
Precedence: bulk

On Mon Oct 20, 2025 at 11:18 AM -03, =C3=81lvaro Herrera wrote:
> On 2025-Oct-20, Matheus Alcantara wrote:
>
>> This is similar to what was already proposed at [1]. This approach was
>> abandoned because a notification on the queue may block datfrozenxid
>> advance and clog truncation which can cause other issues for the users [=
2].
>
> Well, I think that this is the right solution for backpatching, and that
> you were wrong to abandon it.  You can continue to design a better
> mechanism for the master branch, but in old branches we cannot really do
> all those things you're proposing to do.
>=20
I actually would prefer this approach TBH, but since this can cause
other issues like transaction wraparound due to not consumed
notifications we would need other mechanisms to prevent that and I'm not
sure if users should expect this kind of behavior changes on minor
version updates?=20

I think that to go with this solution we would need some way to drop too
old notifications from the queue to advance the datfrozenxid, so I
imagine that we would need some GUC to make this configurable and we can
configure a default value of course but some use cases may not be the
best configuration, this is something that users should expected to deal
on minor version updates?

Going with the "self contained" idea sound more easier to backpatch
actually, so this is the main reason that I abandoned this other
approach. Could you please point what make the v8 version not visible
for bachpatching?

Thanks for the comments!

--
Matheus Alcantara