On Mon, Aug 18, 2014 at 9:37 AM, Peter Eisentraut <pete...@gmx.net> wrote: > It's not ready for committer if the current patch does not apply. FWIW, the latest version sent by Amit here applies correctly: http://www.postgresql.org/message-id/caa4ek1+cc9rb1s9q4+nsokfas1yufvfgfxuhxy_6wlbq1re...@mail.gmail.com I haven't tested it myself though.
However, independently on this patch and as pointed by MauMau, the code that has been committed in fb05f3c is incorrect in the way it defines the tablespace path, this: psprintf("%s/pg_tblspc/%d", basedir, oid); should be this: psprintf("%s/pg_tblspc/%u", basedir, oid); I am separating this fix (that should be backpatched to REL9_4_STABLE as well), in the patch attached if this helps. Regards, -- Michael
diff --git a/src/bin/pg_basebackup/pg_basebackup.c b/src/bin/pg_basebackup/pg_basebackup.c index 3d26e22..ef7cd94 100644 --- a/src/bin/pg_basebackup/pg_basebackup.c +++ b/src/bin/pg_basebackup/pg_basebackup.c @@ -1119,7 +1119,7 @@ update_tablespace_symlink(Oid oid, const char *old_dir) if (strcmp(old_dir, new_dir) != 0) { - char *linkloc = psprintf("%s/pg_tblspc/%d", basedir, oid); + char *linkloc = psprintf("%s/pg_tblspc/%u", basedir, oid); if (unlink(linkloc) != 0 && errno != ENOENT) {
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers