From: | Marti Raudsepp <marti(at)juffo(dot)org> |
---|---|
To: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | [PATCH] pg_upgrade -o/-O regression in 9.2.2 |
Date: | 2012-12-10 22:17:11 |
Message-ID: | CABRT9RBy1uRTMY-DfLDgBPCp+ZQU36M_+BLD4WFWuGOhg-M4hw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi!
It seems that PostgreSQL 9.2.2 has a regression in pg_upgrade, the -o
and -O options forget to add a space before passing on user options,
thereby generating unparsable command lines.
For example:
pg_upgrade -b /usr/local/pg91/bin -B /usr/bin -d /tmp/91 -D /tmp/92 -O -F
[...]
Creating catalog dump ok
*failure*
could not connect to new postmaster started with the command:
"/usr/bin/pg_ctl" -w -l "pg_upgrade_server.log" -D "/tmp/92" -o "-p
50432 -b -c synchronous_commit=off-F -c listen_addresses='' -c
unix_socket_permissions=0700 -c unix_socket_directory='/tmp'" start
Notice the bad argument "synchronous_commit=off-F"
It's easy enough to work around by adding a space to the command line,
passing -O ' -F' instead of -O '-F'
Here's the bad commit:
http://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=ed5699dd1b883e193930448b7ad532e233de0bd7;hp=5ed6546cf75623ba426942a3b71659a66cf7ed68
The attached patch re-introduces the space at the necessary place.
Regards,
Marti
Attachment | Content-Type | Size |
---|---|---|
pg_upgrade_user_options-9.2.patch | application/octet-stream | 964 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2012-12-10 22:18:53 | Re: Support for REINDEX CONCURRENTLY |
Previous Message | Peter Geoghegan | 2012-12-10 21:23:14 | Re: enhanced error fields |