Skip site navigation (1) Skip section navigation (2)

Re: [HACKERS] Resurrecting per-page cleaner for btree

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: pgsql-hackers(at)postgresql(dot)org, ITAGAKI Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp>, pgsql-patches(at)postgresql(dot)org, teramoto(dot)junji(at)lab(dot)ntt(dot)co(dot)jp
Subject: Re: [HACKERS] Resurrecting per-page cleaner for btree
Date: 2006-09-12 04:52:39
Message-ID: 10243.1158036759@sss.pgh.pa.us (view raw or flat)
Thread:
Lists: pgsql-hackerspgsql-patches
Bruce Momjian <bruce(at)momjian(dot)us> writes:
> Tom Lane wrote:
>> I've applied this but I'm now having some second thoughts about it,
>> because I'm seeing an actual *decrease* in pgbench numbers from the
>> immediately prior CVS HEAD code.

> The attached patch requires the new row to fit, and 10% to be free on
> the page.  Would someone test that?

At the moment, I cannot replicate any consistent difference between
CVS head with the patch, without the patch, with the patch plus your
BLCKSZ/10 limit addition, or with a variant BLCKSZ/32 limit addition.
That's whether I use HEAD's broken version of pgbench or one from late
July.  So I'm feeling a tad frustrated ... but I have no evidence in
favor of changing what is in CVS, and accordingly recommend that we
leave well enough alone for 8.2.

			regards, tom lane

In response to

Responses

pgsql-hackers by date

Next:From: Albe LaurenzDate: 2006-09-12 07:03:42
Subject: Re: [PATCHES] Fix linking of OpenLDAP libraries
Previous:From: Alvaro HerreraDate: 2006-09-12 03:21:08
Subject: Re: Buildfarm vs. Linux Distro classification

pgsql-patches by date

Next:From: Dave PageDate: 2006-09-12 10:53:49
Subject: contrib/sslinfo fix
Previous:From: Gregory StarkDate: 2006-09-11 14:43:18
Subject: Some editing of docs for create index concurrently

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group