"Gregory Stark" <[EMAIL PROTECTED]> writes:

> "Tom Lane" <[EMAIL PROTECTED]> writes:
>
>> "Pavan Deolasee" <[EMAIL PROTECTED]> writes:
>>> Please see the version 14 of HOT patch attached.
>>
>> I expected to find either a large new README, or some pretty substantial
>> additions to existing README files, to document how this all works.
>> The comments included do not represent nearly enough documentation.
>
> The Heikki and I posted a two-part README of sorts:

Er, editing error there. Has a ring to it though.

> http://archives.postgresql.org/pgsql-patches/2007-07/msg00142.php
> http://archives.postgresql.org/pgsql-patches/2007-07/msg00360.php
...
>> I also don't think I believe the reasoning for not indexing
>> DELETE_IN_PROGRESS hot-updated tuples: what if the index completion commits,
>> but the concurrent delete rolls back? Then you've got a valid tuple that's
>> not in the index.
>
> You're talking about the concurrent index build case? Wouldn't the second pass
> pick up that tuple? I have to look back at it to see for sure.

Sorry, that's misguided. The concurrent index build uses snapshots now so it
can't see DELETE_IN_PROGRESS. And the non-concurrent index build has an lock
so it ought to be back to the way it was before I messed with it where there
was an assert against finding *_IN_PROGRESS (except as Pavan points out in the
same transaction).

-- 
  Gregory Stark
  EnterpriseDB          http://www.enterprisedb.com

---------------------------(end of broadcast)---------------------------
TIP 1: if posting/reading through Usenet, please send an appropriate
       subscribe-nomail command to [EMAIL PROTECTED] so that your
       message can get through to the mailing list cleanly

Reply via email to