public inbox for [email protected]
help / color / mirror / Atom feedFrom: Zhijie Hou (Fujitsu) <[email protected]>
To: shveta malik <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Cc: Amit Kapila <[email protected]>
Cc: Masahiko Sawada <[email protected]>
Subject: RE: Fix premature xmin advancement during fast forward decoding
Date: Wed, 23 Apr 2025 08:20:53 +0000
Message-ID: <OS0PR01MB571654A01CD98B1870C02F0D94BA2@OS0PR01MB5716.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <CAJpy0uDiozoxBwOb01QVTKvfMZ9GRy4PF1DqC56ULPhEt8_NxQ@mail.gmail.com>
References: <OS0PR01MB57163087F86621D44D9A72BF94BB2@OS0PR01MB5716.jpnprd01.prod.outlook.com>
<CAJpy0uDiozoxBwOb01QVTKvfMZ9GRy4PF1DqC56ULPhEt8_NxQ@mail.gmail.com>
On Wed, Apr 23, 2025 at 2:31 PM shveta malik wrote:
>
> On Tue, Apr 22, 2025 at 12:36 PM Zhijie Hou (Fujitsu)
> <[email protected]> wrote:
> >
> > Hi,
> >
> > To fix this, I think we can allow the base snapshot to be built during fast
> > forward decoding, as implemented in the patch 0001 (We already built base
> > snapshot in fast-forward mode for logical message in logicalmsg_decode()).
>
> The idea and code looks okay to me and the performance impact is also
> not that huge.
Thanks for reviewing!
>
> IIUC, fast_forward decoding mode is used only in two cases. 1)
> pg_replication_slot_advance and 2) in upgrade flow to check if there
> are any pending WAL changes which are not yet replicated. See
> 'binary_upgrade_logical_slot_has_caught_up'-->'LogicalReplicationSlotHasP
> endingWal'.
> It seems like this change will not have any negative impact in the
> upgrade flow as well (in terms of performance and anything else).
> Thoughts?
I also think so. The upgrade uses fast forward decoding to check if there are
any pending WALs not sent yet and report an ERROR if so. This indicates that
we do not expect to decode real changes for the upgrade slots, so the
performance impact would be minimal in normal cases.
Best Regards,
Hou zj
view thread (5+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: RE: Fix premature xmin advancement during fast forward decoding
In-Reply-To: <OS0PR01MB571654A01CD98B1870C02F0D94BA2@OS0PR01MB5716.jpnprd01.prod.outlook.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox