]> granicus.if.org Git - postgresql/commit
Change the way pg_basebackup's tablespace mapping is implemented.
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Fri, 22 Aug 2014 07:16:26 +0000 (10:16 +0300)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Fri, 22 Aug 2014 07:27:50 +0000 (10:27 +0300)
commit8a22633f52f2399118d6ad3c2c4da6a31f07e907
treebfafc117765b1f46a690c73cf35b3a0627e727e3
parentd9b2bc45cf75f913490f1b3ce9b9263509b26704
Change the way pg_basebackup's tablespace mapping is implemented.

Previously, we would first create the symlinks the way they are in the
original system, and at the end replace them with the mapped symlinks.
That never really made much sense, so now we create the symlink pointing
to the correct location to begin with, so that there's no need to fix
them at the end.

The old coding didn't work correctly on Windows, because Windows junction
points look more like directories than files, and ought to be removed with
rmdir rather than unlink. Also, it incorrectly used "%d" rather than "%u"
to print an Oid, but that's gone now.

Report and patch by Amit Kapila, with minor changes by me. Reviewed by
MauMau. Backpatch to 9.4, where the --tablespace feature was added.
src/bin/pg_basebackup/pg_basebackup.c