Re: Debian mips: Failed test 'Check expected t_009_tbl data on standby'

From: Christoph Berg <myon(at)debian(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Debian mips: Failed test 'Check expected t_009_tbl data on standby'
Date: 2018-10-11 19:49:27
Message-ID: 20181011194927.GA31931@msg.df7cb.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Re: Tom Lane 2018-10-11 <28384(dot)1539266025(at)sss(dot)pgh(dot)pa(dot)us>
> Christoph Berg <myon(at)debian(dot)org> writes:
> > The 11rc1 build was failing on Debian mips:
> > ...
> > The diff is that "14|issued to london" was expected, but "|" was
> > received.
> > The build succeeded when I retried it manually on a different box.
>
> Was it repeatable on the original box?

I asked for a rebuild on the buildd network, which succeeded. It wasn't
the very same machine, though. The processor information says
"EdgeRouter Pro" both for the failing build and the manual build, the
3rd (good) build now is "Rhino Labs UTM8".

> My gut feeling is that this is probably a test-case instability
> (eg timing dependency) rather than an actual bug, but of course
> it's hard to be sure.

Looking at src/test/recovery/t/009_twophase.pl, this is testing 2PC,
and the missing row is from:

$cur_master->psql(
'postgres', "
BEGIN;
INSERT INTO t_009_tbl VALUES (13, 'issued to ${cur_master_name}');
SAVEPOINT s1;
INSERT INTO t_009_tbl VALUES (14, 'issued to ${cur_master_name}');
PREPARE TRANSACTION 'xact_009_6';
COMMIT PREPARED 'xact_009_6';");
$cur_master->teardown_node;

So it is very strange that the "13" came through, but "14" did not.
Doesn't look like a test that could be unstable, at least not in that
way.

Christoph

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2018-10-11 20:45:44 Re: Soon-to-be-broken regression test case
Previous Message David Rowley 2018-10-11 19:45:37 Re: Soon-to-be-broken regression test case