RE: [Todo item] Add entry creation timestamp column to pg_stat_replication

From: "myungkyu(dot)lim" <myungkyu(dot)lim(at)samsung(dot)com>
To: "'Michael Paquier'" <michael(at)paquier(dot)xyz>, "'Masahiko Sawada'" <sawada(dot)mshk(at)gmail(dot)com>
Cc: "'PostgreSQL-development'" <pgsql-hackers(at)postgresql(dot)org>, <woosung(dot)sohn(at)samsung(dot)com>, <don(dot)hong(at)samsung(dot)com>
Subject: RE: [Todo item] Add entry creation timestamp column to pg_stat_replication
Date: 2018-11-26 05:48:39
Message-ID: 010201d4854b$ae87e510$0b97af30$@samsung.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Nov 15, 2018 at 5:27 PM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>> Good point. 'last_reply_send_time' is better.
>> How about just 'reply_time'?
>
>Please note that the original thread has mentioned reply_timestamp as a
>consensus:

So I selected 'reply_time' because 'timestamp' was not used in stat field.

>On Thu, Nov 15, 2018 at 05:33:26PM +0900, Masahiko Sawada wrote:
>> Yeah, I also agree with 'reply_time'. But please also note that we had
>> the discussion when there is not the similar system catalogs and
>> fields. Now that we have them it might be worth to consider to follow
>> the existing name for consistency.
>
>The fields in pg_stat_wal_receiver are named after their respective fields.
>Now if you look at the fields from pg_stat_replication, you have those from
>the standby:
>sent_lsn => Last write-ahead log location sent on this connection write_lsn
>=> Last write-ahead log location written to disk by this standby server
>flush_lsn => Last write-ahead log location flushed to disk by this standby
>server replay_lsn => Last write-ahead log location replayed into the
>database on this standby server
>
>So to keep the brand consistent, reply_time looks like the best choice as
>Sawada-san suggests?

I agree.
The fields naming in pg_stat_replication are some different from other views
fields.
Not used 'last_' or 'latest_' prefix in fields.

Best regards,
Myungkyu, Lim

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Langote 2018-11-26 05:57:33 Re: Inadequate executor locking of indexes
Previous Message Laurenz Albe 2018-11-26 05:44:33 Re: Updated backup APIs for non-exclusive backups