Re: Re: [COMMITTERS] pgsql: Send new protocol keepalive messages to standby servers.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Re: [COMMITTERS] pgsql: Send new protocol keepalive messages to standby servers.
Date: 2012-05-24 18:52:05
Message-ID: 27889.1337885525@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> On Wed, May 23, 2012 at 2:28 PM, Fujii Masao <masao(dot)fujii(at)gmail(dot)com> wrote:
>> Is there plan to implement such external functions before 9.2 release?
>> If not, keepalive protocol seems to be almost useless because there is
>> no use of it for a user and the increase in the number of packets might
>> increase the replication performance overhead slightly. No?

> Good point. IMHO, this shouldn't really have been committed like
> this, but since it was, we had better fix it, either by reverting the
> change or forcing an initdb to expose the functionality.

I see no reason to rip the code out if we have plans to make use of it
in the near future. I am also not for going back into development mode
on 9.2, which is what adding new functions now would amount to. What's
wrong with leaving well enough alone? It's not like there is no
unfinished work anywhere else in Postgres ...

regards, tom lane

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2012-05-24 20:11:22 Re: Re: [COMMITTERS] pgsql: Send new protocol keepalive messages to standby servers.
Previous Message Robert Haas 2012-05-24 18:45:51 Re: Re: [COMMITTERS] pgsql: Send new protocol keepalive messages to standby servers.

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2012-05-24 18:54:18 Re: "could not open relation with OID" errors after promoting the standby to master
Previous Message Tom Lane 2012-05-24 18:46:21 Re: "could not open relation with OID" errors after promoting the standby to master