From: | Martin Pihlak <martin(dot)pihlak(at)gmail(dot)com> |
---|---|
To: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Simon Riggs <simon(at)2ndQuadrant(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: reducing statistics write overhead |
Date: | 2009-01-21 16:15:29 |
Message-ID: | 49774A21.1050005@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Alvaro Herrera wrote:
> You missed putting back the BUG comment that used to be there about
> this.
>
This was deliberate, I did mention the condition in the comment at
the beginning of the file. This actually makes it a feature :)
Seriously though, do you think that this is still a problem? Given
the rare occurrence of the revacuum and the fact that it is made
cheap by visibility map? In my initial testing, I couldn't reproduce
the revacuum. But I'll keep at it.
> In other words I think this is a bad idea, because there is a very wide
> window for a table to be vacuumed twice. Since naptime can be
> arbitrarily large, this is an arbitrarily large bug. I'm sure there are
> other ways to fix this, but please propose those before this patch.
>
I was wondering that maybe the stats subsystem shouldn't be used for
vacuum tracking at all. It maybe convenient to use, but has several
deficiencies (pobig file, lossy, no crash safety, etc). Could we move
vacuum tracking to pg_class instead?
regards,
Martin
From | Date | Subject | |
---|---|---|---|
Next Message | Heikki Linnakangas | 2009-01-21 16:24:45 | Re: Pluggable Indexes (was Re: rmgr hooks (v2)) |
Previous Message | Heikki Linnakangas | 2009-01-21 16:11:51 | Re: Pluggable Indexes |