On Thu, 2010-01-21 at 18:01 +0900, Hiroyuki Yamada wrote:
> I think the patch has two problems.
>
> * disable_standby_sig_alarm() does not clear standby_timeout_active flag
> when it succeeds in disabling the alarm.
Ah, thanks.
> * Assertion check in HoldingBufferPinThatDelaysRecovery() can fail
> with following scenario.
Agreed. Will remove Assert() because of race conditions.
Thanks,
-- Simon Riggs www.2ndQuadrant.com