Re: [COMMITTERS] pgsql: Add TCP keepalive support to libpq.

From: Simon Riggs <simon(at)2ndQuadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Florian Pflug <fgp(at)phlo(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com>, Robert Haas <rhaas(at)postgresql(dot)org>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [COMMITTERS] pgsql: Add TCP keepalive support to libpq.
Date: 2010-06-24 17:38:23
Message-ID: 1277401103.25074.8617.camel@ebony
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Thu, 2010-06-24 at 11:15 -0400, Tom Lane wrote:
> Florian Pflug <fgp(at)phlo(dot)org> writes:
> > On Jun 24, 2010, at 16:45 , Robert Haas wrote:
> >> I think the existing names are fine - people should understand that
> >> "options" means "server-side options" and that anything else is a
> >> client-side option. However, if there's a strong consensus the other
> >> way and someone feels like working up a patch, that's fine too.
>
> > I'd volunteer to create the patch if people think renaming the libpq options is a good idea.
>
> I'm with Robert: the names are fine as-is. We've not had complaints
> about the libpq SSL parameters being confusingly like server-side SSL
> parameters, for instance.
>
> It might be a good idea to add a sentence to the documentation, though,
> just pointing out that these control client-side keepalive probes rather
> than server-side.

Yes please. I was confused; I think others will be also.

--
Simon Riggs www.2ndQuadrant.com

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Simon Riggs 2010-06-24 17:40:06 Re: [COMMITTERS] pgsql: Add TCP keepalive support to libpq.
Previous Message Robert Haas 2010-06-24 16:40:45 pgsql: Add stray "else" that seems to have gone missing.

Browse pgsql-hackers by date

  From Date Subject
Next Message Simon Riggs 2010-06-24 17:40:06 Re: [COMMITTERS] pgsql: Add TCP keepalive support to libpq.
Previous Message Josh Berkus 2010-06-24 17:34:09 Re: EOL for 7.4 and 8.0