Re: pg_stat directory and pg_stat_statements - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: pg_stat directory and pg_stat_statements
Date
Msg-id CABUevEwyooQVJquMxzXjhTiyhQ6M1B4rxYSxfpbDNYXiQzYzUQ@mail.gmail.com
Whole thread Raw
In response to Re: pg_stat directory and pg_stat_statements  (Andres Freund <andres@2ndquadrant.com>)
Responses Re: pg_stat directory and pg_stat_statements  (Fujii Masao <masao.fujii@gmail.com>)
List pgsql-hackers
On Mon, Jun 2, 2014 at 4:07 PM, Andres Freund <andres@2ndquadrant.com> wrote:
On 2014-06-02 22:59:55 +0900, Fujii Masao wrote:
> On Thu, May 29, 2014 at 3:02 PM, Peter Geoghegan <pg@heroku.com> wrote:
> > On Wed, May 28, 2014 at 10:49 PM, Fujii Masao <masao.fujii@gmail.com> wrote:
> >> You're concerned about the scenario using pg_upgrade? I'm not sure the detail
> >> of pg_upgrade. But if it doesn't work properly, we should have gotten
> >> the trouble
> >
> > I'm not worried about pg_upgrade, because by design pg_stat_statements
> > will discard stats files that originated in earlier versions. However,
> > I don't see a need to change pg_stat_statements to serialize its
> > statistics to disk in the pg_stat directory before we branch off 9.4.
> > As you mentioned, it's harmless.
>
> Yeah, that's an idea. OTOH, there is no *strong* reason to postpone
> the fix to 9.5. So I just feel inclined to apply the fix now...

+1 for fixing it now.


+1 for fixing it for 9.4 before the next beta, but *not* backpatching it to 9.3 - it *is* a behaviour change after all..

--
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/

pgsql-hackers by date:

Previous
From: Soroosh Sardari
Date:
Subject: Memory deallocation after calling cast function
Next
From: Andres Freund
Date:
Subject: Re: [BUGS] BUG #9652: inet types don't support min/max