Re: Backup history file should be replicated in Streaming Replication? - Mailing list pgsql-hackers

From Fujii Masao
Subject Re: Backup history file should be replicated in Streaming Replication?
Date
Msg-id 3f0b79eb0912232039o5f256eeeqdc7da75d4ecc0de@mail.gmail.com
Whole thread Raw
In response to Re: Backup history file should be replicated in Streaming Replication?  (Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>)
Responses Re: Backup history file should be replicated in Streaming Replication?  (Fujii Masao <masao.fujii@gmail.com>)
List pgsql-hackers
On Wed, Dec 23, 2009 at 7:50 PM, Heikki Linnakangas
<heikki.linnakangas@enterprisedb.com> wrote:
> Ok. How about writing the history file in pg_stop_backup() for
> informational purposes only. Ie. never read it, but rely on the WAL
> records instead.

Sounds good. I'll make such change as a self-contained patch.

> I just realized that the current history file fails to recognize this
> scenario:
>
> 1. pg_start_backup()
> 2. cp -a $PGDATA data-backup
> 3. create data-backup/recovery.conf
> 4. postmaster -D data-backup
>
> That is, starting postmaster on a data directory, without ever calling
> pg_stop_backup(). Because pg_stop_backup() was not called, the history
> file is not there, and recovery won't complain about not reaching the
> safe starting point.
>
> That is of course a case of "don't do that!", but perhaps we should
> refuse to start up if the backup history file is not found? At least in
> the WAL-based approach, I think we should refuse to start up if we don't
> see the pg_stop_backup WAL record.

Agreed.

Regards,

-- 
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center


pgsql-hackers by date:

Previous
From: Fujii Masao
Date:
Subject: updateMinRecoveryPoint bug?
Next
From: Bruce Momjian
Date:
Subject: Re: Removing pg_migrator limitations