[DOC] pg_stat_replication_slots representation style inconsisitant
Hi
When reading the manual doc, I found an inconsistent representation style for the newly added pg_stat_prefetch_recovery view in PG14 at [1]https://www.postgresql.org/docs/devel/monitoring-stats.html#MONITORING-PG-STAT-SUBSCRIPTION.
Same problem seems to be reported by Noriyoshi Shinoda who also proposed a patch to fix the problem at[2]/messages/by-id/TU4PR8401MB1152945E4FD45E99C3F52B94EE4F9@TU4PR8401MB1152.NAMPRD84.PROD.OUTLOOK.COM.
I tried to fix the problem in a way different than Noriyoshi Shinoda which I think is more consistent with the existing specific view introduction in the same page.
Please take the attached patch as your reference.
[1]: https://www.postgresql.org/docs/devel/monitoring-stats.html#MONITORING-PG-STAT-SUBSCRIPTION
https://www.postgresql.org/docs/devel/monitoring-stats.html#MONITORING-PG-STAT-SUBSCRIPTION
[2]: /messages/by-id/TU4PR8401MB1152945E4FD45E99C3F52B94EE4F9@TU4PR8401MB1152.NAMPRD84.PROD.OUTLOOK.COM
/messages/by-id/TU4PR8401MB1152945E4FD45E99C3F52B94EE4F9@TU4PR8401MB1152.NAMPRD84.PROD.OUTLOOK.COM
Regards,
Tang
Attachments:
0001-pg_stat_prefetch_recovery_doc.patchapplication/octet-stream; name=0001-pg_stat_prefetch_recovery_doc.patchDownload+116-69
When reading the manual doc, I found an inconsistent representation style for
the newly added pg_stat_prefetch_recovery view in PG14 at [1].
Same problem seems to be reported by Noriyoshi Shinoda who also proposed
a patch to fix the problem at[2].
I tried to fix the problem in a way different than Noriyoshi Shinoda which I think
is more consistent with the existing specific view introduction in the same
page.
Please take the attached patch as your reference.
I noticed one more thing.
It seems the column " stats_reset | timestamp with time zone " is not listed in the pg_stat_prefetch_recovery view.
Should we add this column too ?
Best regards,
houzj
On Friday, April 30, 2021 12:38 PM, Hou, Zhijie/侯 志杰 <houzj.fnst@fujitsu.com> wrote
I noticed one more thing.
It seems the column " stats_reset | timestamp with time zone " is not listed in the pg_stat_prefetch_recovery view.
Should we add this column too ?
Indeed, column added. Thanks.
Regards,
Tang
Attachments:
0001-pg_stat_prefetch_recovery_doc_v2.patchapplication/octet-stream; name=0001-pg_stat_prefetch_recovery_doc_v2.patchDownload+125-69
On Fri, Apr 30, 2021 at 03:48:27AM +0000, tanghy.fnst@fujitsu.com wrote:
Indeed, column added. Thanks.
I was looking at this thread, and please note that there is no need to
do anything here as the WAL prefetch has been reverted for now as of
c2dc1934.
--
Michael
On Tuesday, May 11, 2021 2:42 PM, Michael Paquier <michael@paquier.xyz> wrote:
I was looking at this thread, and please note that there is no need to
do anything here as the WAL prefetch has been reverted for now as of
c2dc1934.
Thanks for the updating. Got it.
Regards,
Tang
On Wed, May 12, 2021 at 12:49 AM tanghy.fnst@fujitsu.com
<tanghy.fnst@fujitsu.com> wrote:
On Tuesday, May 11, 2021 2:42 PM, Michael Paquier <michael@paquier.xyz> wrote:
I was looking at this thread, and please note that there is no need to
do anything here as the WAL prefetch has been reverted for now as of
c2dc1934.Thanks for the updating. Got it.
Thanks for the feedback. I'll be re-proposing the feature for 15 and
I've made a note about this thread.