Re: unnecessary code in_bt_split - Mailing list pgsql-hackers

From Zdenek Kotala
Subject Re: unnecessary code in_bt_split
Date
Msg-id 489879CD.2000306@sun.com
Whole thread Raw
In response to Re: unnecessary code in_bt_split  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Tom Lane napsal(a):
> Zdenek Kotala <Zdenek.Kotala@Sun.COM> writes:

> I'm thinking we should split PageGetTempPage into two versions:
> 
> PageGetTempPage: get a temp page the same size as the given page,
> but don't initialize its contents at all (so, just a thin wrapper
> for palloc).  This could be used by _bt_split, as well as
> GinPageGetCopyPage and GistPageGetCopyPage.
> 
> PageGetTempPageCopySpecial: get a temp page, PageInit it, and
> copy the special space from the given page.  The only customer
> for this is gistplacetopage(), so maybe we don't even want it,
> rather than just doing the work right in gistplacetopage()?
> 
> You could also make an argument for PageGetTempPageCopy() which'd just
> copy the source page verbatim, thus replacing GinPageGetCopyPage and
> GistPageGetCopyPage.

Sounds good I will look on it.
    Zdenek


-- 
Zdenek Kotala              Sun Microsystems
Prague, Czech Republic     http://sun.com/postgresql



pgsql-hackers by date:

Previous
From: Gregory Stark
Date:
Subject: Re: Status of DISTINCT-by-hashing work
Next
From: Markus Wanner
Date:
Subject: Re: Automatic Client Failover