]> 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:21 +0000 (10:27 +0300)
commit8d938eb0797e7acd925a2adf72315f8ab1586d83
tree4dbf8e695f879bbf8bc0831d68f8c40a6bdd8687
parent3feda1bcbb653e954092bddd9c3eb681ab51fa2b
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