Re: [PATCHES] VACUUM Improvements - WIP Patch

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Gregory Stark <stark(at)enterprisedb(dot)com>
Cc: "Pavan Deolasee" <pavan(dot)deolasee(at)gmail(dot)com>, "Pgsql Hackers" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCHES] VACUUM Improvements - WIP Patch
Date: 2008-07-14 17:55:00
Message-ID: 28739.1216058100@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Gregory Stark <stark(at)enterprisedb(dot)com> writes:
> I like the idea of only having to do a single pass through the table though.

Well, that argument was already overstated: we're not re-reading all of
the table now. Just the pages containing dead line pointers.

> Couldn't Pavan's original plan still work and just not have other clients try
> to remove dead line pointers?

You could simply delay recycling of the really-truly-dead line pointers
until the next VACUUM, I suppose. It's not clear how bad a
line-pointer-bloat problem that might leave you with. (It would still
require tracking whether the last vacuum had completed successfully.
I note that any simple approach to that would foreclose ever doing
partial-table vacuums, which is something I thought was on the table
as soon as we had dead space mapping ability.)

> At least not unless they're also pruning the
> page due to an insert or update anyways?

Please stop pretending that this overhead will only be paid by
insert/update. The current design for pruning does not work that way,
and we do not have a better design available.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrew Dunstan 2008-07-14 17:55:07 Re: PATCH: CITEXT 2.0 v3
Previous Message David E. Wheeler 2008-07-14 17:51:21 Re: testing locales and encodings

Browse pgsql-patches by date

  From Date Subject
Next Message Gregory Stark 2008-07-14 18:24:14 Re: [PATCHES] VACUUM Improvements - WIP Patch
Previous Message Gregory Stark 2008-07-14 17:15:42 Re: [PATCHES] VACUUM Improvements - WIP Patch