On Tue, Oct 08, 2019 at 02:18:00AM +0900, Fujii Masao wrote: > On Fri, Oct 4, 2019 at 9:03 PM Fujii Masao <masao.fu...@gmail.com> wrote: >> So archive recovery with recovery_min_apply_delay enabled would be >> intended configuration. My patch that changes archive recovery so that >> it always ignores thesetting might be in wrong direction. Maybe we should >> make recovery_min_apply_delay work properly even in archive recovery. >> Thought? > > Patch attached. This patch allows archive recovery with > recovery_min_apply_delay set, but not crash recovery.
Right. In short it makes no sense to wait the delay when in crash recovery. After more testing I have been able to reproduce the failure myself. + /* nothing to do if crash recovery is requested */ + if (!ArchiveRecoveryRequested && !StandbyModeRequested) + return false; ArchiveRecoveryRequested will be set to true if recovery.signal or standby.signal are found, so it seems to me that you can make all those checks more simple by removing from the equation StandbyModeRequested, no? StandbyModeRequested is never set to true if ArchiveRecoveryRequested is not itself true. It would be nice to test some scenario within 002_archiving.pl. -- Michael
signature.asc
Description: PGP signature