Re: [PATCH] Make pg_basebackup configure and start standby

On Wed, Oct 3, 2012 at 10:37 AM, Boszormenyi Zoltan <zb(at)cybertec(dot)at> wrote:
> Hi,
>
> 2012-10-03 10:25 keltezéssel, Magnus Hagander írta:
>>
>> On Tue, Jul 3, 2012 at 9:47 PM, Peter Eisentraut <peter_e(at)gmx(dot)net> wrote:
>>>
>>> On mĺn, 2012-07-02 at 01:10 +0900, Fujii Masao wrote:
>>>>>
>>>>> 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?
>>>>
>>>> What about something like PQconninfo which returns the connection
>>>> string value established at connection?
>>>>
>>>>> 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?
>>>>
>>>> +1
>>>>
>>> I think both of these would be necessary to make this work smoothly.
>>>
>>> You also need to take into account situations like when pg_basebackup
>>> found a server with the help of PG* environment variables that might no
>>> longer be set when the server tries to start recovery. So you need
>>> something like PQconninfo.
>>
>> Zoltan,
>>
>> are you planning to work on the things discussed in this thread? I
>> notice the patch is sitting with "waiting on author" in the CF app -
>> so the second question is that if you are doing that, do you think it
>> will be done within the scope of the current CF?
>
>
> yes, I am on it so it can be done in this CF.

Great, thanks!

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