Re: pgbench regression test failure

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
Cc: PostgreSQL Developers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pgbench regression test failure
Date: 2017-09-22 17:02:25
Message-ID: 13924.1506099745@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> writes:
>> It could be as simple as putting the check-for-done at the bottom of the
>> loop not the top, perhaps.

> I agree that it is best if tests should work in all reasonable conditions,
> including a somehow overloaded host...

> I'm going to think about it, but I'm not sure of the best approach. In the
> mean time, ISTM that the issue has not been encountered (yet), so this is
> not a pressing issue. Maybe under -T > --aggregate-interval pgbench could
> go on over the limit if the log file has not been written at all, but that
> would be some kind of kludge for this specific test...

After another week of buildfarm runs, we have a few more cases of 3 rows
of output, and none of more than 3 or less than 1. So I went ahead and
pushed your patch. I'm still suspicious of these results, but we might
as well try to make the buildfarm green pending investigation of how
this is happening.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2017-09-22 17:21:09 Re: additional contrib test suites
Previous Message Melanie Plageman 2017-09-22 16:57:07 Re: [PATCH v1] Add and report the new "in_hot_standby" GUC pseudo-variable.