From 9ee49afdf9ad7e53152658b23ae8d2dd55bb3194 Mon Sep 17 00:00:00 2001 From: Shinya Kato Date: Thu, 20 Nov 2025 14:56:05 +0900 Subject: [PATCH v8 2/2] Add started_by column to pg_stat_progress_analyze. Exposes the reason why the current ANALYZE operation was initiated (manual or autovacuum). This allows users and monitoring tools to better understand ANALYZE behavior. Bump catalog version. Author: Shinya Kato Discussion: https://postgr.es/m/CAA5RZ0suoicwxFeK_eDkUrzF7s0BVTaE7M%2BehCpYcCk5wiECpw%40mail.gmail.com --- doc/src/sgml/monitoring.sgml | 25 +++++++++++++++++++++++++ src/backend/catalog/system_views.sql | 5 ++++- src/backend/commands/analyze.c | 6 ++++++ src/include/commands/progress.h | 5 +++++ src/test/regress/expected/rules.out | 7 ++++++- 5 files changed, 46 insertions(+), 2 deletions(-) diff --git a/doc/src/sgml/monitoring.sgml b/doc/src/sgml/monitoring.sgml index 7836e028f8f..44d8bdb0fc9 100644 --- a/doc/src/sgml/monitoring.sgml +++ b/doc/src/sgml/monitoring.sgml @@ -5748,6 +5748,31 @@ FROM pg_stat_get_backend_idset() AS backendid; zero). + + + + started_by text + + + Shows what caused the current ANALYZE operation to be + started. Possible values are: + + + + manual: The analyze was started by an explicit + ANALYZE or VACUUM (ANALYZE) + command. + + + + + autovacuum: The analyze was started by an + autovacuum worker. + + + + + diff --git a/src/backend/catalog/system_views.sql b/src/backend/catalog/system_views.sql index 8fc56bccc1f..910ef0aa2e2 100644 --- a/src/backend/catalog/system_views.sql +++ b/src/backend/catalog/system_views.sql @@ -1244,7 +1244,10 @@ CREATE VIEW pg_stat_progress_analyze AS S.param6 AS child_tables_total, S.param7 AS child_tables_done, CAST(S.param8 AS oid) AS current_child_table_relid, - S.param9 / 1000000::double precision AS delay_time + S.param9 / 1000000::double precision AS delay_time, + CASE S.param10 WHEN 1 THEN 'manual' + WHEN 2 THEN 'autovacuum' + ELSE NULL END AS started_by FROM pg_stat_get_progress_info('ANALYZE') AS S LEFT JOIN pg_database D ON S.datid = D.oid; diff --git a/src/backend/commands/analyze.c b/src/backend/commands/analyze.c index 25089fae3e0..b2429170679 100644 --- a/src/backend/commands/analyze.c +++ b/src/backend/commands/analyze.c @@ -239,6 +239,12 @@ analyze_rel(Oid relid, RangeVar *relation, */ pgstat_progress_start_command(PROGRESS_COMMAND_ANALYZE, RelationGetRelid(onerel)); + if (AmAutoVacuumWorkerProcess()) + pgstat_progress_update_param(PROGRESS_ANALYZE_STARTED_BY, + PROGRESS_ANALYZE_STARTED_BY_AUTOVACUUM); + else + pgstat_progress_update_param(PROGRESS_ANALYZE_STARTED_BY, + PROGRESS_ANALYZE_STARTED_BY_MANUAL); /* * Do the normal non-recursive ANALYZE. We can skip this for partitioned diff --git a/src/include/commands/progress.h b/src/include/commands/progress.h index 45ed6e6e1cc..9dc63a5a5bd 100644 --- a/src/include/commands/progress.h +++ b/src/include/commands/progress.h @@ -60,6 +60,7 @@ #define PROGRESS_ANALYZE_CHILD_TABLES_DONE 6 #define PROGRESS_ANALYZE_CURRENT_CHILD_TABLE_RELID 7 #define PROGRESS_ANALYZE_DELAY_TIME 8 +#define PROGRESS_ANALYZE_STARTED_BY 9 /* Phases of analyze (as advertised via PROGRESS_ANALYZE_PHASE) */ #define PROGRESS_ANALYZE_PHASE_ACQUIRE_SAMPLE_ROWS 1 @@ -68,6 +69,10 @@ #define PROGRESS_ANALYZE_PHASE_COMPUTE_EXT_STATS 4 #define PROGRESS_ANALYZE_PHASE_FINALIZE_ANALYZE 5 +/* Reasons for analyze (as advertised via PROGRESS_ANALYZE_STARTED_BY) */ +#define PROGRESS_ANALYZE_STARTED_BY_MANUAL 1 +#define PROGRESS_ANALYZE_STARTED_BY_AUTOVACUUM 2 + /* Progress parameters for cluster */ #define PROGRESS_CLUSTER_COMMAND 0 #define PROGRESS_CLUSTER_PHASE 1 diff --git a/src/test/regress/expected/rules.out b/src/test/regress/expected/rules.out index f8e40e2d99c..180a2265e95 100644 --- a/src/test/regress/expected/rules.out +++ b/src/test/regress/expected/rules.out @@ -1968,7 +1968,12 @@ pg_stat_progress_analyze| SELECT s.pid, s.param6 AS child_tables_total, s.param7 AS child_tables_done, (s.param8)::oid AS current_child_table_relid, - ((s.param9)::double precision / (1000000)::double precision) AS delay_time + ((s.param9)::double precision / (1000000)::double precision) AS delay_time, + CASE s.param10 + WHEN 1 THEN 'manual'::text + WHEN 2 THEN 'autovacuum'::text + ELSE NULL::text + END AS started_by FROM (pg_stat_get_progress_info('ANALYZE'::text) s(pid, datid, relid, param1, param2, param3, param4, param5, param6, param7, param8, param9, param10, param11, param12, param13, param14, param15, param16, param17, param18, param19, param20) LEFT JOIN pg_database d ON ((s.datid = d.oid))); pg_stat_progress_basebackup| SELECT pid, -- 2.47.3