From: | "Pavan Deolasee" <pavan(dot)deolasee(at)gmail(dot)com> |
---|---|
To: | "Heikki Linnakangas" <heikki(at)enterprisedb(dot)com> |
Cc: | "Jaime Casanova" <systemguards(at)gmail(dot)com>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Maintaining cluster order on insert |
Date: | 2007-05-21 08:17:06 |
Message-ID: | 2e78013d0705210117y6a65245cw426bbaafa9eb6419@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
On 5/19/07, Heikki Linnakangas <heikki(at)enterprisedb(dot)com> wrote:
>
>
> Ah, sorry about that. For some reason my source tree was checked out
> from the 8.2 branch, instead of CVS HEAD.
>
>
I looked at the patch. Not that I am very comfortable with this part
of the code, but nevertheless here are my comments:
I am wondering if we can optimize _bt_moveright() code by remembering
the PageLSN of the page before releasing the READ LOCK. If the page
is indeed changed since we last saw it, the PageLSN must be different
than what we had remembered and gives us a hint that we might need
to move right.
! /* Page was full. Release lock and pin and get another block
! * as if suggested_blk was not given.
! */
! LockBuffer(suggested_buf, BUFFER_LOCK_UNLOCK);
! ReleaseBuffer(suggested_buf);
May be you want to use UnlockReleaseBuffer() or _bt_relbuf() just as a
shorthand.
Btw, I wonder why _bt_relbuf() exists and even so why does it take
"Relation"
argument ?
Thanks,
Pavan
--
EnterpriseDB http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2007-05-21 09:00:50 | [Fwd: PGBuildfarm member vaquita Branch HEAD Status changed from ECPG-Check failure to OK] |
Previous Message | NikhilS | 2007-05-21 06:53:42 | Re: CREATE TABLE LIKE INCLUDING INDEXES support |
From | Date | Subject | |
---|---|---|---|
Next Message | Pavan Deolasee | 2007-05-21 09:32:18 | Re: Maintaining cluster order on insert |
Previous Message | Guillaume Lelarge | 2007-05-21 07:43:56 | Patch needed fot dt.h |