On 18.01.2013 13:41, Amit Kapila wrote:
> On Friday, January 18, 2013 3:46 PM Heikki Linnakangas wrote:
>> On 18.01.2013 08:50, Amit Kapila wrote:
>>> I think currently user has no way to specify TCP keepalive settings
>> from
>>> pg_basebackup, please let me know if there is any such existing way?
>>
>> I was going to say you can just use "keepalives_idle=30" in the
>> connection string. But there's no way to pass a connection string to
>> pg_basebackup on the command line! The usual way to pass a connection
>> string is to pass it as the database name, and PQconnect will expand
>> it,
>> but that doesn't work with pg_basebackup because it hardcodes the
>> database name as "replication". D'oh.
>>
>> You could still use environment variables and a service file to do it,
>> but it's certainly more cumbersome. It clearly should be possible to
>> pass a full connection string to pg_basebackup, that's an obvious
>> oversight.
>
> So to solve this problem below can be done:
> 1. Support connection string in pg_basebackup and mention keepalives or
> connection_timeout
> 2. Support recv_timeout separately to provide a way to users who are not
> comfortable tcp keepalives
>
> a. 1 can be done alone
> b. 2 can be done alone
> c. both 1 and 2.
Right. Let's do just 1 for now. An general application level, non-TCP,
keepalive message at the libpq level might be a good idea, but that's a
much larger patch, definitely not 9.3 material.
- Heikki
