From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Dylan Hansen <dhansen(at)pixpo(dot)com>, "Matthew T(dot) O'Connor" <matthew(at)zeut(dot)net>, pgsql-general(at)postgresql(dot)org |
Subject: | Re: auto-vacuum & Negative "anl" Values |
Date: | 2006-06-27 12:42:36 |
Message-ID: | 20060627124236.GA14418@surnet.cl |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general |
Tom Lane wrote:
> Alvaro Herrera <alvherre(at)commandprompt(dot)com> writes:
> > + /* last_anl_tuples must never exceed n_live_tuples */
>
> If we actually believe the above statement, it seems like your patch
> to pgstat_recv_tabstat() opens a new issue: with that patch, it is
> possible for pgstat_recv_tabstat() to decrease n_live_tuples, and
> therefore a clamp needs to be applied in pgstat_recv_tabstat() too.
> No?
Hmm, yeah.
> The reason I didn't patch it myself is that I'm not quite clear on what
> *should* be happening here. What effect should a large delete have on
> the ANALYZE threshold, exactly? You could argue that a deletion
> potentially changes the statistics (by omission), and therefore inserts,
> updates, and deletes should equally count +1 towards the analyze
> threshold. I don't think we are implementing that though. If we want
> to do it that way, I suspect last_anl_tuples as currently defined is not
> the right comparison point.
Maybe what we should do is revert the pgstat_recv_tabstat() part of the
patch in 8.1, and consider redefining last_anl_tuples in HEAD. Caffeine
is not high enough yet to propose anything sensible, but I'll think
about it a bit later.
--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support
From | Date | Subject | |
---|---|---|---|
Next Message | dananrg | 2006-06-27 12:51:07 | Re: Best open source tool for database design / ERDs? |
Previous Message | surabhi.ahuja | 2006-06-27 12:22:19 | planning to upgrade to 8.1 |