public inbox for pgsql-hackers@postgresql.org
help / color / mirror / Atom feedFrom: Michael Paquier <michael@paquier.xyz>
To: Chao Li <li.evan.chao@gmail.com>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Cc: Michael Paquier <michael.paquier@gmail.com>
Cc: Xuneng Zhou <xunengzhou@gmail.com>
Subject: Re: Fix pg_stat_wal_receiver to show CONNECTING status
Date: Thu, 21 May 2026 05:43:52 +0900
Message-ID: <ag4dCGAPBc5VFhCi@paquier.xyz> (raw)
In-Reply-To: <75CDE990-29D5-4D5C-BFE1-3840F19C0163@gmail.com>
References: <EF91FF76-1E2B-4F3B-9162-290B4DC517FF@gmail.com>
<agxr29Hsz7FjxzlN@paquier.xyz>
<1F153E64-B791-42FA-A60A-64813B20B81E@gmail.com>
<ag00OeH1sbt5ie_6@paquier.xyz>
<75CDE990-29D5-4D5C-BFE1-3840F19C0163@gmail.com>
On Wed, May 20, 2026 at 03:53:38PM +0800, Chao Li wrote:
> With v2, slot_name, sender_host, sender_port, and conninfo are
> already left NULL while the receiver is in CONNECTING state. I feel
> we don't have to show the timestamp fields either. Since the columns
> are named last_msg_send_time and last_msg_receipt_time, users may
> naturally interpret them as the last time a message was sent to or
> received from
> the primary. If we show the standby server start time in those
> columns, I am afraid that could be confusing.
>
> But I think it might be useful to show the *_lsn and *_tli values in
> CONNECTING state if they are available.
The original reason why ready_to_display has been introduced is this
one, where we wanted to have a strict control over the connection
information across multiple calls of pg_stat_get_wal_receiver():
https://www.postgresql.org/message-id/CAB7nPqQNbHQ7F7wDD_2qvGA_FUW-Leds9HQNM6kJnto7RFNhUg@mail.gmail...
With v2, ready_to_display is still able to do the job it is defined
for. This does not need to apply on the time fields, so IMO showing
them to the values they are initialized is not a big deal, and they
can actually be useful to know even in the early stage of connection
as they reveal the state of the code.
Note also that the time values could still show up based on their
initial values at the early connection stage, even after completing
walrcv_connect() and after ready_to_display is switched to true, so
it's not like these values are that confusing: we just expose them a
bit more at an earlier stage of the connection attempt process. As a
whole v2 is fine, and addresses your issue.
--
Michael
Attachments:
[application/pgp-signature] signature.asc (833B, 2-signature.asc)
download
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: michael@paquier.xyz, li.evan.chao@gmail.com, michael.paquier@gmail.com, xunengzhou@gmail.com
Subject: Re: Fix pg_stat_wal_receiver to show CONNECTING status
In-Reply-To: <ag4dCGAPBc5VFhCi@paquier.xyz>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox