Re: Bug with pg_basebackup and 'shared' tablespace

From: Pierre Ducroquet <p(dot)psql(at)pinaraf(dot)info>
To: <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Bug with pg_basebackup and 'shared' tablespace
Date: 2017-05-01 17:28:59
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On Friday, April 7, 2017 3:12:58 AM CEST, Kyotaro HORIGUCHI wrote:
> Hi, Pierre.
> Maybe you're the winner:p
> At Thu, 06 Apr 2017 12:34:09 +0200, Pierre Ducroquet
> <p(dot)psql(at)pinaraf(dot)info> wrote in <1714428(dot)BHRm6e8A2D(at)peanuts2>
>> On Thursday, April 6, 2017 2:00:55 PM CEST Kyotaro HORIGUCHI wrote: ...
> That being said, it is a different matter if the behavior is
> preferable. The discussion on the behavior is continued here.
>> Right now, there is a conflict between pg_basebackup and the
>> server since they
>> do not allow the same behaviour. I can submit a patch either
>> way, but I won't
>> decide what is the right way to do it.
>> I know tricks will allow to work around that issue, I found
>> them hopefully and
>> I guess most people affected by this issue would be able to
>> find and use them,
>> but nevertheless being able to build a server that can no longer be base-
>> backuped does not seem right.
> regards,


I didn't have much time to spend on that issue until today, and I found a
way to solve it that seems acceptable to me.

The biggest drawback will be that if the backup is interrupted, previous
tablespaces already copied will stay on disk, but since that issue could
already happen, I don't think it's too much a drawback.
The patch simply delays the empty folder checking until we start reading
the tablespace tarball. The first entry of the tarball being the
PG_MAJOR_CATVER folder, that can not be found otherwise, there is no real
alternative to that.

I will submit this patch in the current commit fest.



Attachment Content-Type Size
0001-Allow-a-pg_basebackup-when-a-tablespace-is-shared-be.patch text/x-patch 3.0 KB

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2017-05-01 17:36:53 Re: PQhost may return socket dir for network connection
Previous Message Kevin Grittner 2017-05-01 17:17:03 Re: transition table behavior with inheritance appears broken (was: Declarative partitioning - another take)