Re: pg_basebackup creates a corrupt file for pg_stat_tmp and pg_replslot on a backup location - Mailing list pgsql-hackers

From Ashutosh Sharma
Subject Re: pg_basebackup creates a corrupt file for pg_stat_tmp and pg_replslot on a backup location
Date
Msg-id CAE9k0P=eTy9n0rCC0xO1Cf6TZTFRCLURo9OJJKeo2mhCgy0ULA@mail.gmail.com
Whole thread Raw
In response to Re: pg_basebackup creates a corrupt file for pg_stat_tmp and pg_replslot on a backup location  (Magnus Hagander <magnus@hagander.net>)
Responses Re: pg_basebackup creates a corrupt file for pg_stat_tmp and pg_replslot on a backup location  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
List pgsql-hackers
Hi,

Knowing that pg_basebackup always creates an empty directory for pg_stat_tmp and pg_replslot in backup location, even i think it would be better to handle these directories in such a way that pg_basebackup generates an empty directory for pg_replslot and pg_stat_tmp if they are symbolic link.

PFA patch for the same.

With Regards,
Ashutosh Sharma
EnterpriseDB: http://www.enterprisedb.com

On Thu, Apr 14, 2016 at 11:57 PM, Magnus Hagander <magnus@hagander.net> wrote:
On Thu, Apr 14, 2016 at 8:20 PM, Ashutosh Sharma <ashu.coek88@gmail.com> wrote:
Hi,

I was just curious to know how would "pg_basebackup" behave if we do create a symbolic link for directories other than pg_xlog/pg_tblspc. However it is clearly mentioned in the documentation of pg_basebackup that if a  Symbolic link for the directories other than pg_tblspc and pg_xlog is created then it would be skipped. But, that is not the case for pg_replslot and pg_stat_tmp. Is this not an issue. Should these directories not be skipped. Please let me know your thoughts on this. Thanks.

I agree that actually generating a corrupt tarfile is not good. But I think the correct fix is to actually generate an empty placeholder directory rather than skipping it - thereby making the backup look the same as it would if it was a correct directory where we just skipped the contents.

--

Attachment

pgsql-hackers by date:

Previous
From: Erik Rijkers
Date:
Subject: Re: Declarative partitioning
Next
From: Dean Rasheed
Date:
Subject: Re: [COMMITTERS] pgsql: Add trigonometric functions that work in degrees.