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
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 |
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 |