On Fri, May 01, 2020 at 03:02:59AM +0900, Fujii Masao wrote:


On 2020/04/02 9:41, Tomas Vondra wrote:
Hi,

I've pushed this after some minor cleanup and improvements.

+static char *slru_names[] = {"async", "clog", "commit_timestamp",
+                                                         "multixact_offset", 
"multixact_member",
+                                                         "oldserxid", "pg_xact", 
"subtrans",
+                                                         "other" /* has to be 
last */};

When I tried pg_stat_slru, I found that it returns a row for "pg_xact".
But since there is no "pg_xact" slru ("clog" slru exists instead),
"pg_xact" should be removed? Patch attached.


Yeah, I think I got confused and accidentally added both "clog" and
"pg_xact". I'll get "pg_xact" removed.

Regards,

--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

diff --git a/doc/src/sgml/monitoring.sgml b/doc/src/sgml/monitoring.sgml
index 6562cc400b..ba6d8d2123 100644
--- a/doc/src/sgml/monitoring.sgml
+++ b/doc/src/sgml/monitoring.sgml
@@ -3483,7 +3483,7 @@ SELECT pid, wait_event_type, wait_event FROM 
pg_stat_activity WHERE wait_event i
       predefined list (<literal>async</literal>, <literal>clog</literal>,
       <literal>commit_timestamp</literal>, <literal>multixact_offset</literal>,
       <literal>multixact_member</literal>, <literal>oldserxid</literal>,
-       <literal>pg_xact</literal>, <literal>subtrans</literal> and
+       <literal>subtrans</literal> and
       <literal>other</literal>) resets counters for only that entry.
       Names not included in this list are treated as <literal>other</literal>.
      </entry>
diff --git a/src/backend/postmaster/pgstat.c b/src/backend/postmaster/pgstat.c
index 50eea2e8a8..2ba3858d31 100644
--- a/src/backend/postmaster/pgstat.c
+++ b/src/backend/postmaster/pgstat.c
@@ -152,7 +152,7 @@ PgStat_MsgBgWriter BgWriterStats;
 */
static char *slru_names[] = {"async", "clog", "commit_timestamp",
                                                          "multixact_offset", 
"multixact_member",
-                                                         "oldserxid", "pg_xact", 
"subtrans",
+                                                         "oldserxid", 
"subtrans",
                                                          "other" /* has to be 
last */};

/* number of elemenents of slru_name array */


--
Tomas Vondra                  http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


Reply via email to