Thread: [PATCH] hs_standby_disallowed test fix

[PATCH] hs_standby_disallowed test fix

From
Michail Nikolaev
Date:
Hello.

There is a recent commit about changes in way read-only commands are
prevented to be executed [1].

It seems like hs_standby_disallowed test is broken now.

So, a simple patch to fix the test is attached.

Thanks,
Michail.

[1]
https://www.postgresql.org/message-id/flat/154701965766.11631.2240747476287499810.pgcf%40coridan.postgresql.org#168075c6e89267e11b862aa0c55b1910

Attachment

Re: [PATCH] hs_standby_disallowed test fix

From
Fujii Masao
Date:

On 2020/05/12 8:03, Michail Nikolaev wrote:
> Hello.
> 
> There is a recent commit about changes in way read-only commands are
> prevented to be executed [1].
> 
> It seems like hs_standby_disallowed test is broken now.
> 
> So, a simple patch to fix the test is attached.

Thanks for the report and patch! LGTM.

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.

Regards,

-- 
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION



Re: [PATCH] hs_standby_disallowed test fix

From
Fujii Masao
Date:

On 2020/05/12 12:05, Fujii Masao wrote:
> 
> 
> On 2020/05/12 8:03, Michail Nikolaev wrote:
>> Hello.
>>
>> There is a recent commit about changes in way read-only commands are
>> prevented to be executed [1].
>>
>> It seems like hs_standby_disallowed test is broken now.
>>
>> So, a simple patch to fix the test is attached.
> 
> Thanks for the report and patch! LGTM.

Pushed. Thanks!

Regards,

-- 
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION



Re: [PATCH] hs_standby_disallowed test fix

From
Tom Lane
Date:
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.

            regards, tom lane



Re: [PATCH] hs_standby_disallowed test fix

From
Peter Eisentraut
Date:
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