Re: [PATCH] Make pg_basebackup configure and start standby - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: [PATCH] Make pg_basebackup configure and start standby
Date
Msg-id CABUevEwEma9FT1yjJa7q_ow4Ce28qfb-J+P8tSf1npKn_umOeQ@mail.gmail.com
Whole thread Raw
In response to [PATCH] Make pg_basebackup configure and start standby  (Boszormenyi Zoltan <zb@cybertec.at>)
Responses Re: [PATCH] Make pg_basebackup configure and start standby
Re: [PATCH] Make pg_basebackup configure and start standby
List pgsql-hackers
On Sun, Jul 1, 2012 at 1:02 PM, Boszormenyi Zoltan <zb@cybertec.at> wrote:
> Hi,
>
> attached is a patch that does $SUBJECT.
>
> It's a usability enhancement, to take a backup, write
> a minimalistic recovery.conf and start the streaming
> standby in one go.

I like the writing of recovery.conf. In fact, I had it in my code at
one very early point and took it out in order to get a clean patch
ready :)

But I think that part is lacking in functionality: AFAICT it's
hardcoded to only handle host, port, user and password. What about
other connection parameters, likely passed to pg_basebackup through
the environment in that case? isn't that quite likely to break the
server later?

Maybe the proper way around that is to provide the ability for
pg_basebackup to take a full connection string, just like we allow
psql to do?



I'm not sure we should go the way of providing the "start slave".
Given thta how you want to start the slave differs so much on
platforms. The most glaring example is on windows you really need to
*start the service* rather than use pg_ctl. Sure, you can document
your way around that, but I'm not sure the functionality added is
really worth it. What about all the other potential connection
parameters?


--
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/


pgsql-hackers by date:

Previous
From: Boszormenyi Zoltan
Date:
Subject: Re: [PATCH] Make pg_basebackup configure and start standby
Next
From: Alexander Korotkov
Date:
Subject: Re: [PATCH 14/16] Add module to apply changes from an apply-cache using low-level functions