Re: [HACKERS] What does it mean by XLOG_BACKUP_RECORD?

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