From 4a347206fa406e9e4a28fc44482b9b487173b535 Mon Sep 17 00:00:00 2001 From: Wang Hao Date: Fri, 12 Jul 2019 18:22:52 +0800 Subject: [PATCH] Skip start the stats sender process when gpperfmon not enabled (#8126) Before GP6, the stats sender process works for gpperfmon and metrics collector. If any of them enabled, stats sender should start. From GP6 metrics collector become a standalone bgworker. So this commit fix startup condition for stats sender, now it only starts when gpperfmon enabled. --- src/backend/postmaster/perfmon_segmentinfo.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/backend/postmaster/perfmon_segmentinfo.c b/src/backend/postmaster/perfmon_segmentinfo.c index 00e9bd919c..445bec40f5 100644 --- a/src/backend/postmaster/perfmon_segmentinfo.c +++ b/src/backend/postmaster/perfmon_segmentinfo.c @@ -87,9 +87,7 @@ void SegmentInfoSenderMain(Datum main_arg) bool SegmentInfoSenderStartRule(Datum main_arg) { - /* - * TODO: is this correct? either GUC is on will turn on this auxiliary process ? */ - if (!gp_enable_gpperfmon && !gp_enable_query_metrics) + if (!gp_enable_gpperfmon) return false; /* FIXME: even for the utility mode? */ -- GitLab