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

From: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
To: Simon Riggs <simon(at)2ndquadrant(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Re: [COMMITTERS] pgsql: Send new protocol keepalive messages to standby servers.
Date: 2012-01-13 04:18:40
Message-ID: CAHGQGwGaL73yjLd_S-eyjGOKDdSZnusb+nmxYxxih8NkHneKVw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Fri, Jan 13, 2012 at 4:19 AM, Simon Riggs <simon(at)2ndquadrant(dot)com> wrote:
> On Thu, Jan 12, 2012 at 10:37 AM, Fujii Masao <masao(dot)fujii(at)gmail(dot)com> wrote:
>> On Thu, Jan 12, 2012 at 5:53 PM, Simon Riggs <simon(at)2ndquadrant(dot)com> wrote:
>>> PrimaryKeepaliveMessage is a message type that uses WalSndrMessage.
>>> That message type is only sent when the WalSndr is quiet, so what is
>>> the difference, in that case?
>>
>> Oh, you are right. There is no difference.
>
> Here are the changes we discussed. Further comments before commit?

Can you add the test for avoiding useless call of GetReplicationApplyDelay()
and GetReplicationTransferLatency()?

Regards,

--
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Simon Riggs 2012-01-13 13:01:05 pgsql: Minor but necessary improvements to WAL keepalives
Previous Message Alex Hunsaker 2012-01-13 02:28:22 Re: [COMMITTERS] pgsql: Fix breakage from earlier plperl fix.

Browse pgsql-hackers by date

  From Date Subject
Next Message YAMAMOTO Takashi 2012-01-13 04:24:12 gistVacuumUpdate
Previous Message Nick Roosevelt 2012-01-13 04:07:04 Postgres ReviewFest Patch: URI connection string support for libpq