On Sun, Jul 1, 2012 at 1:02 PM, Boszormenyi Zoltan <z...@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/ -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers