pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Amit Kapila <akapila@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Fix test failure caused by commit 76b78721ca.
Date: Wed, 26 Nov 2025 03:36:54 +0000
Message-ID: <E1vO6Ks-001RGo-1D@gemulon.postgresql.org> (raw)

Fix test failure caused by commit 76b78721ca.

The test failed because it assumed that a newly created logical
replication slot could be synced to the standby by the slotsync worker.
However, the presence of an existing physical slot caused the new logical
slot to use a non-latest xmin. On the standby, the DDL had already been
replayed, advancing xmin, which led to the slotsync worker failing to sync
the lagging logical slot.

To resolve this, we moved the slot sync statistics tests to run after the
tests that do not require the newly created slot to be sync-ready.

As per buildfarm.

Author: Hou Zhijie <houzj.fnst@fujitsu.com>
Reviewed-by: Hayato Kuroda <kuroda.hayato@fujitsu.com>
Reviewed-by: Amit Kapila <amit.kapila16@gmail.com>
Discussion: https://postgr.es/m/OSCPR01MB14966FE0BFB6C212298BFFEDEF5D1A@OSCPR01MB14966.jpnprd01.prod.outlook.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/e3e787ca023449127ac370e13a3887fe03eed0f4

Modified Files
--------------
.../recovery/t/040_standby_failover_slots_sync.pl  | 128 +++++++++++----------
1 file changed, 70 insertions(+), 58 deletions(-)



Message-ID: <E1vO6Ks-001RGo-1D@gemulon.postgresql.org>
Permalink:  ../E1vO6Ks-001RGo-1D@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1vO6Ks-001RGo-1D@gemulon.postgresql.org

 · 

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: pgsql-committers@postgresql.org
  Cc: akapila@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Fix test failure caused by commit 76b78721ca.
  In-Reply-To: <E1vO6Ks-001RGo-1D@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox