Re: pg_stat_replication vs StandbyReplyMessage

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Simon Riggs <simon(at)2ndquadrant(dot)com>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pg_stat_replication vs StandbyReplyMessage
Date: 2011-08-16 14:02:17
Message-ID: CABUevEwS8T5UOiss+qS_w74DuO4e0Rj0x=XWALamezU2F70kgA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Aug 16, 2011 at 16:00, Simon Riggs <simon(at)2ndquadrant(dot)com> wrote:
> On Tue, Aug 16, 2011 at 2:25 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>> On Tue, Aug 16, 2011 at 8:54 AM, Simon Riggs <simon(at)2ndquadrant(dot)com> wrote:
>>> On Tue, Aug 16, 2011 at 10:51 AM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>>>> On Mon, Aug 15, 2011 at 13:50, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>>>>> On Mon, Aug 15, 2011 at 7:03 AM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>>>>>> The pg_stat_replication view exposes all the fields in
>>>>>> StandbyReplyMessage *except* for the timestamp when the message was
>>>>>> generated. On an active system this is not all that interesting, but
>>>>>> on a mostly idle system that allows the monitoring to react faster
>>>>>> than the timeout that actually kicks the other end off - and could be
>>>>>> useful in manual debugging scenarios. Any particular reason why this
>>>>>> was not exposed as it's own column?
>>>>>
>>>>> I wondered the same thing.  Sounds like a good idea.
>>>>
>>>> I can go do that. Care to argue^Wbikeshed for a specific name?
>>>
>>> reply_timestamp
>>
>> Works for me.
>
>> I'd suggest that we rename it that way in
>> StandbyReplyMessage, so that the name in the struct and the name in
>> the system view match.
>
> -1
>
> The field is named same as equivalent field in other messages.
>
> The field on the view is a summary of all previous messages, which is
> a different thing. Perhaps we should call it last_reply_timestamp to
> make that clearer, though long titles are annoying.

We don't say last_replay_location either, we just say replay_location.
Adding the last_ part is just annoying.

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

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2011-08-16 14:12:49 Re: non-ipv6 vs hostnames
Previous Message Simon Riggs 2011-08-16 14:00:09 Re: pg_stat_replication vs StandbyReplyMessage