On Thu, Nov 29, 2018 at 05:43:26PM +0900, myungkyu.lim wrote:
> Changed field name 'last_msg_send_time' to 'reply_time'.
Looks pretty to me at quick glance, unfortunately I have not spent much
time on it, particularly testing it.
+ <row>
+ <entry><structfield>reply_time</structfield></entry>
+ <entry><type>timestamp with time zone</type></entry>
+ <entry>Send time of last message received from WAL
receiver</entry>
+ </row>
This is a bit confusing as this counts for the last *standby* message
('r'), and not the last feedback message ('h').
+ /*
+ * timestamp of the latest message, reported by standby server
+ */
+ TimestampTz replyTime;
A small indentation problem, not a big deal.
Sawada-san, perhaps you would like to look at it?
--
Michael