Re: [PATCH] hs_standby_disallowed test fix

Peter Eisentraut <peter.eisentraut@2ndquadrant.com>

From: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
To: Tom Lane <tgl@sss.pgh.pa.us>, Fujii Masao <masao.fujii@oss.nttdata.com>
Cc: Michail Nikolaev <michail.nikolaev@gmail.com>, PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2020-05-18T14:23:23Z
Lists: pgsql-hackers
On 2020-05-12 19:35, Tom Lane wrote:
> Fujii Masao <masao.fujii@oss.nttdata.com> writes:
>> I just wonder why standbycheck regression test doesn't run by default
>> in buildfarm. Which caused us not to notice this issue long time. Maybe
>> because it's difficult to set up hot-standby environment in the
>> regression test? If so, we might need to merge standbycheck test into
>> TAP tests for recovery.
> 
> It seems likely to me that the standbycheck stuff has been completely
> obsoleted by the TAP-based recovery tests.  We should get rid of it,
> after adding any missing coverage to the TAP tests.

I have looked into this a few times.  It should definitely be done, but 
there is actually a fair amount of coverage in the standbycheck that is 
not in a TAP test, so it would be a fair amount of careful leg work to 
get this all moved over.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



Commits

  1. Correct standbycheck regression test output.