Re: [BUGS] BUG #1707: statistics collector starts with stats_start_collector

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Alvaro Herrera <alvherre(at)surnet(dot)cl>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Federico Di Gregorio <fog(at)initd(dot)org>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [BUGS] BUG #1707: statistics collector starts with stats_start_collector
Date: 2005-06-28 02:39:55
Message-ID: 2156.1119926395@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-patches

Alvaro Herrera <alvherre(at)surnet(dot)cl> writes:
> What about arranging postmaster's main loop so that it starts the stats
> collector process if one of the options is activated (assuming they were
> all off at start)? Right now you have to restart the server, but I
> don't see why it has to be like that.

Actually, this entire thread is based on a misconception: the "bug"
reporter was under the misimpression that the sum total of what the
collector does is described by the three suboptions, and so having
start_collector on without any of the suboptions on is a useless state.
But this is not so. The collector tracks backend start/end in any case,
and so you get a useful list of sessions in pg_stat_activity even
without enabling any of the more expensive reporting options.

So my vote at this point is that the code is not broken and we should
not change it. Possibly the documentation could stand a little
improvement.

regards, tom lane

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2005-06-28 06:05:33 Re: [BUGS] BUG #1707: statistics collector starts with stats_start_collector
Previous Message satish reddy 2005-06-28 01:35:38 Reg No Error Information Available

Browse pgsql-patches by date

  From Date Subject
Next Message Michael Fuhr 2005-06-28 02:49:18 Re: Performance analysis of plpgsql code
Previous Message Karl O. Pinc 2005-06-28 01:54:08 Re: Performance analysis of plpgsql code