On Thu, Jun 11, 2015 at 9:55 AM, Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Wed, Jun 10, 2015 at 12:09 PM, Fujii Masao <masao.fu...@gmail.com> wrote: > > > > On Tue, Jun 9, 2015 at 3:29 PM, Amit Kapila <amit.kapil...@gmail.com> wrote: > > > On Tue, Jun 9, 2015 at 10:56 AM, Fujii Masao <masao.fu...@gmail.com> wrote: > > >> Or what about removing tablespace_map file at the beginning of recovery > > >> whenever backup_label doesn't exist? > > > > > > Yes, thats another way, but is it safe to assume that user won't need > > > that file, > > > > Is there really case where tablespace_map is necessary even though backup_label > > doesn't exist? If not, it seems safe to get rid of the file when backup_label > > is not present. > > > > > I mean in the valid scenario (where both backup_label and > > > tablespace_map are present and usable) also, we rename them to > > > *.old rather than deleting it. > > > > Yep, I'm OK to make the recovery rename the file to *.old rather than delete it. > > > > This sounds safe to me, unless anybody else has different opinion > I will write a patch to fix this way. >
Attached patch provides a fix as per above discussion. With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com
rename_mapfile_if_backupfile_not_present_v1.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers