Re: [GENERAL] Stats Collector

> Bruce Momjian <pgman@candle.pha.pa.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