Re: [HACKERS] [PATCH] Tap test support for backup with tablespacemapping - Mailing list pgsql-hackers

From Tels
Subject Re: [HACKERS] [PATCH] Tap test support for backup with tablespacemapping
Date
Msg-id 538b2352c4e55cd7ef9764ab28d7923b.squirrel@sm.webmail.pair.com
Whole thread Raw
In response to [HACKERS] [PATCH] Tap test support for backup with tablespace mapping  (Vaishnavi Prabakaran <vaishnaviprabakaran@gmail.com>)
List pgsql-hackers
Dear Vaishnavi,

On Tue, December 26, 2017 8:58 pm, Vaishnavi Prabakaran wrote:
> Hi All,
>
> I have added support in Postgres TAP test framework to backup a data
> directory with tablespace mapping. Also added support to move the backup
> directory contents to standby node, because current option to init the
> standby from backup does not support copying softlinks, which is needed
> when tablespace mapping is involved in backup.
>
> Added a new test to existing streaming replication tap test to demonstrate
> the usage of these new APIs.
>
> Attached the patch, Hope this enhancement is useful.
>
> Thanks & Regards,
> Vaishnavi,
> Fujitsu Australia.
>

Thank you for the path, I saw these things:

* backup_withtablespace() does not have any documentation?

* The mkdir calls do not set a mask for the created dir, defaulting to
0777 - is this what is wanted here?

* none of the mkdir, chdir etc. calls check any error code, e.g. what
happens if one of them fails?

* different indentation between rmdir and move (tab vs. spaces):

+        rmdir($data_path);
+        move("$backup_path", "$self->{_basedir}/pgdata")

Best regards,

Tels


pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: [table partitioning] How many partitions are possibel?
Next
From: Arthur Zakirov
Date:
Subject: Re: [PROPOSAL] Shared Ispell dictionaries