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. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services