Re: pgsql: Introduce replication slots.

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Thom Brown <thom(at)linux(dot)com>
Cc: Andres Freund <andres(at)2ndquadrant(dot)com>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, Robert Haas <rhaas(at)postgresql(dot)org>, pgsql-committers <pgsql-committers(at)postgresql(dot)org>
Subject: Re: pgsql: Introduce replication slots.
Date: 2014-02-25 16:25:38
Message-ID: CA+Tgmobm07+fXZmy4Xsipq1FLR6+nYUZJXSQhu8756s7MVxJtA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Tue, Feb 25, 2014 at 10:11 AM, Thom Brown <thom(at)linux(dot)com> wrote:
> On 6 February 2014 10:29, Andres Freund <andres(at)2ndquadrant(dot)com> wrote:
>> Hi Fujii,
>>
>> On 2014-02-01 16:47:47 +0900, Fujii Masao wrote:
>> > This patch changed basebackup.c so that it skips pg_replslot. It's OK
>> > to skip all files in that directory, but an empty pg_replslot must be
>> > included in the backup. Otherwise we cannot start PostgreSQL from
>> > the backup taken via pg_basebackup. Attached patch fixes this problem.
>>
>> Do you plan to commit this patch? It's clearly an improvement over the
>> current situation...
>
> Yeah, this is still an annoyance.

Since Fujii Masao doesn't seem to be around, I'll commit this. But
don't we need to update the documentation as well, for those using the
file-copy method of taking a backup?

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2014-02-25 16:32:35 pgsql: pg_basebackup: Skip only the *contents* of pg_replslot.
Previous Message Thom Brown 2014-02-25 15:11:55 Re: pgsql: Introduce replication slots.