From: Tom Lane <tgl@sss.pgh.pa.us>
To: Alvaro Herrera <alvherre@alvh.no-ip.org>
cc: pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Fix "base" snapshot handling in logical decoding
In-reply-to: <E1fXuuZ-0004SB-Ue@gemulon.postgresql.org>
References: <E1fXuuZ-0004SB-Ue@gemulon.postgresql.org>
Comments: In-reply-to Alvaro Herrera <alvherre@alvh.no-ip.org>
	message dated "Tue, 26 Jun 2018 20:50:03 -0000"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <26019.1530244389.1@sss.pgh.pa.us>
Content-Transfer-Encoding: quoted-printable
Date: Thu, 28 Jun 2018 23:53:10 -0400
Message-ID: <26020.1530244390@sss.pgh.pa.us>
Precedence: bulk

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.

friarbird's error output doesn't seem to be getting captured by the
buildfarm script, but what I see is

***************
*** 16,21 ****
--- 16,22 ----
  =

  step s1_commit: COMMIT;
  step s0_vacuum: VACUUM FULL;
+ ERROR:  canceling statement due to user request
  step s0_get_changes: SELECT data FROM pg_logical_slot_get_changes('isola=
tion_s
lot', NULL, NULL, 'include-xids', '0', 'skip-empty-xacts', '1');
  data           =

  =

which looks like it might just be a timeout problem.

			regards, tom lane