Re: pg_upgrade: allow multiple -o/-O options

Поиск
Список
Период
Сортировка
От Pavel Raiskup
Тема Re: pg_upgrade: allow multiple -o/-O options
Дата
Msg-id 2837578.WU4QXfJeMN@nb.usersys.redhat.com
обсуждение исходный текст
Ответ на Re: pg_upgrade: allow multiple -o/-O options  (Bruce Momjian <bruce@momjian.us>)
Ответы Re: pg_upgrade: allow multiple -o/-O options  (Bruce Momjian <bruce@momjian.us>)
Список 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




В списке pgsql-hackers по дате отправления:

Предыдущее
От: Amit Kapila
Дата:
Сообщение: Re: option -T in pg_basebackup doesn't work on windows
Следующее
От: Dave Page
Дата:
Сообщение: Re: Are postgresql-9.4 binaries available on Windows XP?