Re: pgsql: Fix a violation of WAL coding rules in the recent patch to

From: Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>
To: Tom Lane <tgl(at)postgresql(dot)org>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Fix a violation of WAL coding rules in the recent patch to
Date: 2009-08-24 06:12:57
Message-ID: 4A922F69.9040402@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Tom Lane wrote:
> --- 74,80 ----
> * That might actually be OK for the index scans, though. The newly inserted
> * tuple wouldn't have an index pointer yet, so all tuples reachable from an
> * index would still be visible to all other backends, and deletions wouldn't
> ! * be visible to other backends yet. (But HOT breaks that argument, no?)

The same argument holds for HOT. The new tuple has no index pointer, so
it's not directly reachable from the index. Moreover, the value in the
index is correct for both tuple versions, because a HOT update doesn't
change index keys.

In fact, if we only maintained the visibility map to enable index-only
scans, a HOT update wouldn't need to clear the bit in the visibility map
at all.

--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2009-08-24 14:15:09 pgsql: Remove unused variable.
Previous Message Heikki Linnakangas 2009-08-24 05:39:45 Re: pgsql: Fix a violation of WAL coding rules in the recent patch to