Re: [GENERAL] Stats Collector

> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> >> It all works now and I have just submitted it to -patches as a
> new contrib,
> >> but it probably should make its way into the backend one day.
>
> > OK, the big question is how do we want to make stats reset visible to
> > users? The current patch uses a function call. Is that how we want to
> > do it?
>
> Should we make it visible at all? I'm concerned about security.

The function it's calling in the backend:

void
pgstat_reset_counters(void)
{
PgStat_MsgResetcounter msg;

if (pgStatSock < 0)
return;

if (!superuser())
elog(ERROR, "Only database superusers can reset statistic
counters");

pgstat_setheader(&msg.m_hdr, PGSTAT_MTYPE_RESETCOUNTER);
pgstat_send(&msg, sizeof(msg));
}

Note it does actually check that you're a superuser...

Chris