Re: BF animal dikkop reported a failure in 035_standby_logical_decoding

Andres Freund <andres@anarazel.de>

From: Andres Freund <andres@anarazel.de>
To: "Drouvot, Bertrand" <bertranddrouvot.pg@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, "Yu Shi (Fujitsu)" <shiy.fnst@fujitsu.com>, "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>, tomas.vondra@enterprisedb.com
Date: 2023-05-30T15:24:26Z
Lists: pgsql-hackers
Hi,

On 2023-05-29 14:31:24 +0200, Drouvot, Bertrand wrote:
> On 5/29/23 1:03 PM, Tom Lane wrote:
> > but I wouldn't be surprised if something in the logical replication
> > mechanism itself could be running a transaction at the wrong instant.
> > 
> > Some of the other recovery tests set
> > autovacuum = off
> > to try to control such problems, but I'm not sure how much of
> > a solution that really is.
> 
> One option I can think of is to:
> 
> 1) set autovacuum = off (as it looks like the usual suspect).
> 2) trigger the vacuum in verbose mode (as suggested by Shi-san) and
> depending of its output run the "invalidation" test or: re-launch the vacuum, re-check the output
> and so on.. (n times max). If n is reached, then skip this test.

I think the best fix would be to wait for a new snapshot that has a newer
horizon, before doing the vacuum full.

Greetings,

Andres Freund



Commits

  1. Improve stability of recovery test 035_standby_logical_decoding