Re: Extend documentation for pg_stat_replication.backend_xmin - Mailing list pgsql-hackers

From Renzo Dani
Subject Re: Extend documentation for pg_stat_replication.backend_xmin
Date
Msg-id CA+XOKQCjH2ZcnRdVDtg-KoASmDxz2ZqU2XSuU9Grups-ngw-Qg@mail.gmail.com
Whole thread Raw
In response to Re: Extend documentation for pg_stat_replication.backend_xmin  (Fujii Masao <masao.fujii@gmail.com>)
Responses Re: Extend documentation for pg_stat_replication.backend_xmin
List pgsql-hackers
Really nice, 
Thanks a lot for the info and review.
Taken into the patch.

Regards,
Renzo Dani

On Thu, Oct 16, 2025 at 5:32 AM Fujii Masao <masao.fujii@gmail.com> wrote:
On Thu, Oct 16, 2025 at 12:47 AM Renzo Dani <arons7@gmail.com> wrote:
>
> Hi,
> recently I had some issue understanding what was going on with a replica and the reported xmin.
> I have than created a patch to better specify the value expected into the pg_stat_replication.backend_xmin field.

+1

+       This field will be null if a replication slot is used; in that
case, the xmin is available through <link
linkend="view-pg-replication-slots">pg_replication_slots</link>.

I think it would be better to also mention that this field is null
when hot_standby_feedback is disabled.

Also, <literal> tags should be used for "xmin", and a newline should
be added to avoid an overly long line.

Based on the above comments, how about revising it as follows?

+       This field will be null if a replication slot is used (in this case,
+       the standby's <literal>xmin</literal> is shown in
+       <link linkend="view-pg-replication-slots">pg_replication_slots</link>)
+       or if <varname>hot_standby_feedback</varname> is disabled.

Regards,

--
Fujii Masao
Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: remove partColsUpdated
Next
From: Michael Paquier
Date:
Subject: Re: Preserve index stats during ALTER TABLE ... TYPE ...