Re: pgsql: Fix "base" snapshot handling in logical decoding - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: pgsql: Fix "base" snapshot handling in logical decoding
Date
Msg-id 20180629225055.cy3e4ecayibwr5rd@alvherre.pgsql
Whole thread Raw
In response to Re: pgsql: Fix "base" snapshot handling in logical decoding  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Responses Re: pgsql: Fix "base" snapshot handling in logical decoding
List pgsql-hackers
On 2018-Jun-29, Alvaro Herrera wrote:

> On 2018-Jun-28, Tom Lane wrote:
> 
> > Alvaro Herrera <alvherre@alvh.no-ip.org> writes:
> > > Fix "base" snapshot handling in logical decoding
> > 
> > According to buildfarm member friarbird, and as confirmed here,
> > the contrib/test_decoding/specs/oldest_xmin.spec test added by this
> > commit fails under CLOBBER_CACHE_ALWAYS.
> 
> Hm.  Running this test under CLOBBER_CACHE_ALWAYS I see that the VACUUM
> FULL step takes 31 seconds (and the test succeeds).  This is not
> top-of-the-line hardware by a long shot (Intel(R) Core(TM) i7-4600U CPU
> @ 2.10GHz) but I can believe that the other machines are slower or
> busier.

It does fail in the indicated way in CLOBBER_CACHE_RECURSIVELY, but I
guess that's expected.  It also fails if I reduce the timeout from 60
seconds to 25.

I suppose 60 seconds (isolationtester's default timeout) is just not
enough time for those machines.  We could increase it to 180 seconds and
see if that's enough to make them pass ...

Another possibility is to examine column 'state' in isolationtester.  I
thought we only used that timeout for 'waiting' queries, but I see it's
not doing that.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


pgsql-hackers by date:

Previous
From: Srinivas Karthik V
Date:
Subject: Re: Bulk Insert into PostgreSQL
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Fix "base" snapshot handling in logical decoding