On Wednesday, March 2, 2022 5:29 PM Shi, Yu/侍 雨 <shiy.fnst@fujitsu.com> wrote:
> A comments on the v26 patch.
Thank you for checking the patch !
>
> The following document about pg_stat_subscription_stats view only says that
> "showing statistics about errors", should we add something about transactions
> here?
>
> <row>
>
> <entry><structname>pg_stat_subscription_stats</structname><indexterm
> ><primary>pg_stat_subscription_stats</primary></indexterm></entry>
> <entry>One row per subscription, showing statistics about errors.
> See <link linkend="monitoring-pg-stat-subscription-stats">
> <structname>pg_stat_subscription_stats</structname></link> for
> details.
> </entry>
> </row>
>
>
> I noticed that the v24 patch has some changes about the description of this
> view. Maybe we can modify to "showing statistics about errors and
> transactions".
You are right. Fixed.
New patch v27 that incorporated your comments is shared in [1].
Kindly have a look at it.
[1] -
https://www.postgresql.org/message-id/TYCPR01MB837345FB72E3CE9C827AF42CED049%40TYCPR01MB8373.jpnprd01.prod.outlook.com
Best Regards,
Takamichi Osumi
