Received: from localhost (unknown [200.46.204.187])
	by postgresql.org (Postfix) with ESMTP id C54B49FA5C9
	for <pgsql-hackers-postgresql.org@postgresql.org>;
	Thu, 13 Sep 2007 00:39:33 -0300 (ADT)
Received: from postgresql.org ([200.46.204.71])
	by localhost (mx1.hub.org [200.46.204.187]) (amavisd-maia, port 10024)
	with ESMTP id 17699-05 for
	<pgsql-hackers-postgresql.org@postgresql.org>;
	Thu, 13 Sep 2007 00:39:25 -0300 (ADT)
X-Greylist: domain auto-whitelisted by SQLgrey-1.7.5
Received: from rv-out-0910.google.com (rv-out-0910.google.com
 [209.85.198.188])
	by postgresql.org (Postfix) with ESMTP id 567249FA510
	for <pgsql-hackers@postgresql.org>;
	Thu, 13 Sep 2007 00:39:25 -0300 (ADT)
Received: by rv-out-0910.google.com with SMTP id l15so247857rvb
	for <pgsql-hackers@postgresql.org>;
	Wed, 12 Sep 2007 20:39:24 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=beta;
	h=domainkey-signature:received:received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:references;
	bh=xuwLyk5zTuVrn5UJtwiAExRZdvBY0KV6KfQmJjKMfAc=;
	b=pN9tTG9D/lMfVaaBAge8jMW0lGfkviAVgvJhcAGHXn30WB0F8jA8epItII2bY/FYFsyutF54wJP2G1mdAfDNtUUPD2cd73CQFWGxW24Kkf+rRQ5dWh2w6BKthnmLW7FItW8j3+/Wl4UpeJU4pYGJtY3Q9SzbDEqcZcxQgwY/EQM=
DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta;
	h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:references;
	b=Tlm5r1C1fwGAyaKaj2g2CghueJIMRxNFwa7TaxMcthXsQRTSowhZ+3TJCoJNQCScCVXx1/Q9qA11gC9ShTPz2XcxSaIlUfVtgZGrWKOkWCejoxOJ5ZjE+WeBN2kplOmj6Sf0QNtZze2LJCDFfQH7XzQcxXt7ZuxniOeyP5tINC4=
Received: by 10.141.23.7 with SMTP id a7mr33533rvj.1189654764295;
	Wed, 12 Sep 2007 20:39:24 -0700 (PDT)
Received: by 10.141.87.11 with HTTP; Wed, 12 Sep 2007 20:39:24 -0700 (PDT)
Message-ID: <2e78013d0709122039p30625abage94ce9d6fd3bbcd2@mail.gmail.com>
Date: Thu, 13 Sep 2007 09:09:24 +0530
From: "Pavan Deolasee" <pavan.deolasee@gmail.com>
To: "Tom Lane" <tgl@sss.pgh.pa.us>
Subject: Re: Representation of redirected line pointers in HOT
Cc: pgsql-hackers@postgresql.org
In-Reply-To: <25006.1189621802@sss.pgh.pa.us>
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----=_Part_17175_14480076.1189654764258"
References: <25006.1189621802@sss.pgh.pa.us>
X-Virus-Scanned: Maia Mailguard 1.0.1
X-Archive-Number: 200709/593
X-Sequence-Number: 107745

------=_Part_17175_14480076.1189654764258
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

On 9/13/07, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
>
> What I'm thinking is that we should instead do this by extending the use
> of the lp_flags field.  lp_flags is two bits, which we currently define
> as independent LP_USED and LP_DELETE bits, but in fact LP_DELETE is
> never used in heap pages.  (It is used in indexes.)  I propose that
> we redefine lp_flags as having four states, say
>
>         LP_UNUSED       0
>         LP_NORMAL       1
>         LP_REDIRECT     2
>         LP_DEAD         3
>
>
Sounds good to me. I saw you committed this change. Do you want me
to update HOT patch to use this or you are already doing that ?


Thanks,
Pavan

-- 
Pavan Deolasee
EnterpriseDB     http://www.enterprisedb.com

------=_Part_17175_14480076.1189654764258
Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

<br><br><div><span class="gmail_quote">On 9/13/07, <b class="gmail_sendername">Tom Lane</b> &lt;<a href="mailto:tgl@sss.pgh.pa.us">tgl@sss.pgh.pa.us</a>&gt; wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>What I&#39;m thinking is that we should instead do this by extending the use<br>of the lp_flags field.&nbsp;&nbsp;lp_flags is two bits, which we currently define<br>as independent LP_USED and LP_DELETE bits, but in fact LP_DELETE is
<br>never used in heap pages.&nbsp;&nbsp;(It is used in indexes.)&nbsp;&nbsp;I propose that<br>we redefine lp_flags as having four states, say<br><br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;LP_UNUSED&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 0<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;LP_NORMAL&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 1<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;LP_REDIRECT&nbsp;&nbsp;&nbsp;&nbsp; 2<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;LP_DEAD&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 3
<br><br></blockquote></div><br>Sounds good to me. I saw you committed this change. Do you want me<br>to update HOT patch to use this or you are already doing that ?<br><br><br>Thanks,<br>Pavan<br><br>-- <br>Pavan Deolasee
<br>EnterpriseDB&nbsp;&nbsp;&nbsp;&nbsp; <a href="http://www.enterprisedb.com">http://www.enterprisedb.com</a>

------=_Part_17175_14480076.1189654764258--