From: | Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Zdenek Kotala <Zdenek(dot)Kotala(at)Sun(dot)COM>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: FSM patch - performance test |
Date: | 2008-09-18 18:04:40 |
Message-ID: | 48D29838.9040405@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Tom Lane wrote:
> Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> writes:
>> Zdenek Kotala wrote:
>>> My conclusion is that new implementation is about 8% slower in OLTP
>>> workload.
>
>> Thanks. That's very disappointing :-(
>
> One thing that jumped out at me is that you call FreeSpaceMapExtendRel
> every time a rel is extended by even one block. I admit I've not
> studied the data structure in any detail yet, but surely most such calls
> end up being a no-op? Seems like some attention to making a fast path
> for that case would be helpful.
Yes, most of those calls end up being no-op. Which is exactly why I
would be surprised if those made any difference. It does call
smgrnblocks(), though, which isn't completely free...
Zdenek, can you say off the top of your head whether the test was I/O
bound or CPU bound? What was the CPU utilization % during the test?
--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Gregory Stark | 2008-09-18 18:19:34 | Re: Do we really need a 7.4.22 release now? |
Previous Message | Tom Lane | 2008-09-18 18:04:03 | Re: New FSM patch |