On Sat, Dec 9, 2017 at 1:25 AM, Peter Eisentraut <peter.eisentr...@2ndquadrant.com> wrote: > On 6/29/17 06:09, Masahiko Sawada wrote: >> Thanks, I agree to use XLOG_BACKUP_END instead. >> >>> Worse, the current comment implies that >>> minRecoveryPoint is incorrectly set if it is true. Bleh. >> >> Looking at the condition, we use minRecoveryPoint only when >> ControlFile->backupEndRequired is *false*. So I guess that it means >> that minRecoveryPoint is incorrectly set if >> ControlFile->backupEndReuired is true. Am I missing something? > > I agree with you that the logic in the comment is correct. I've > committed just the symbol change. >
Thank you for picking up an old thread and committing it! Regards, -- Masahiko Sawada NIPPON TELEGRAPH AND TELEPHONE CORPORATION NTT Open Source Software Center