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
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. |