On 2019-09-29 18:36, Fujii Masao wrote: > Yes, but ArchiveRecoveryRequested should be checked instead of > InArchiveRecovery, I think. Otherwise recovery targets would take effect > when recovery.signal is missing but backup_label exists. In this case, > InArchiveRecovery is set to true though ArchiveRecoveryRequested is > false because recovery.signal is missing. > > With the attached patch, I checked that the steps that I described > upthread didn't reproduce the issue.
Your patch looks correct to me. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
