From: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> |
---|---|
To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: What does it mean by XLOG_BACKUP_RECORD? |
Date: | 2017-06-29 10:09:19 |
Message-ID: | CAD21AoDt2oBBaF1Cvn3AUagVBX27RnF_A+qtka8oz-p6xLxWNA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Jun 29, 2017 at 10:49 AM, Michael Paquier
<michael(dot)paquier(at)gmail(dot)com> wrote:
> On Thu, Jun 29, 2017 at 10:28 AM, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
>> While reading source codes I found the following comment in xlog.c.
>>
>> /*
>> * Have we passed our safe starting point? Note that minRecoveryPoint is
>> * known to be incorrectly set if ControlFile->backupEndRequired, until
>> * the XLOG_BACKUP_RECORD arrives to advise us of the correct
>> * minRecoveryPoint. All we know prior to that is that we're not
>> * consistent yet.
>> */
>> if (!reachedConsistency && !ControlFile->backupEndRequired &&
>> minRecoveryPoint <= lastReplayedEndRecPtr &&
>> XLogRecPtrIsInvalid(ControlFile->backupStartPoint))
>>
>> What does XLOG_BACKUP_RECORED means by? I could not find such XLOG info value.
>> Does it mean XLOG_BACKUP_END?
>
> This comment is a thinko, it refers to XLOG_BACKUP_END. This comment
> block could be reworded a bit, it looks cleaner to me to say
> "ControlFile->backupEndRequired is false" instead of just referring to
> the variable itself.
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?
Regards,
--
Masahiko Sawada
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center
From | Date | Subject | |
---|---|---|---|
Next Message | Etsuro Fujita | 2017-06-29 11:20:41 | Add support for tuple routing to foreign partitions |
Previous Message | Craig Ringer | 2017-06-29 09:39:58 | Re: Start logical decoding from any lsn present in the wal segment |