From: | Pavel Raiskup <praiskup(at)redhat(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org, hhorak(at)redhat(dot)com |
Subject: | Re: pg_upgrade: allow multiple -o/-O options |
Date: | 2014-08-22 08:52:12 |
Message-ID: | 2837578.WU4QXfJeMN@nb.usersys.redhat.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thursday 21 of August 2014 18:26:37 Bruce Momjian wrote:
> On Tue, Mar 4, 2014 at 04:52:56PM +0100, Pavel Raiskup wrote:
> > RFE: Consider that you want to run pg_upgrade via some script with some
> > default '-o' option. But then you also want to give the script's user a
> > chance to specify the old-server's options according user's needs.
> > Then something like the following is not possible:
> >
> > $ cat script
> > ...
> > pg_upgrade ... -o 'sth' $PG_UPGRADE_OPT ...
> > ...
> >
> > I know that this problem is still script-able, but the fix should be
> > innocent and it would simplify things. Thanks for considering,
>
> Attached is a patch that makes multiple -o options append their
> arguments for pg_upgrade and pg_ctl, and documents this and the append
> behavior of postmaster/postgres. This covers all the -o behaviors.
Thanks! Seems to be OK to me, one nit - why you did not go the
append_optiton way (there could be probably better name like arg_cat)?
Because this is just about few lines, it is probably OK from PostgreSQL
policy POV, so "review? ~> review+", thanks again!
Pavel
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2014-08-22 09:11:06 | Re: Are postgresql-9.4 binaries available on Windows XP? |
Previous Message | Amit Kapila | 2014-08-22 08:35:15 | Re: option -T in pg_basebackup doesn't work on windows |