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
-
Correct standbycheck regression test output.
- 81ec990a23b9 13.0 landed