Re: [ADMIN] reindexdb hangs - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: [ADMIN] reindexdb hangs
Date
Msg-id 20070910220700.GC3060@alvh.no-ip.org
Whole thread Raw
In response to Re: [ADMIN] reindexdb hangs  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [ADMIN] reindexdb hangs  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Tom Lane wrote:
> Alvaro Herrera <alvherre@commandprompt.com> writes:
> > I'm not sure I follow.  Are you suggesting adding a new function,
> > similar to pg_class_ownercheck, which additionally checks for temp-ness?
>
> No, I was just suggesting adding the check for temp-ness in cluster()
> and cluster_rel() where we do pg_class_ownercheck.  We already have the
> rel open there and so it's cheap to do the temp-ness check.

I applied a patch along these lines to HEAD and 8.2.

I am unsure if I should backpatch to 8.1: the code in cluster.c has
changed, and while it is relatively easy to modify the patch, this is a
rare bug and nobody has reported it in CLUSTER (not many people clusters
temp tables, it seems).  Should I patch only REINDEX?  How far back?

--
Alvaro Herrera                  http://www.amazon.com/gp/registry/5ZYLFMCVHXC
"The eagle never lost so much time, as
when he submitted to learn of the crow." (William Blake)

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: ispell dictionary broken in CVS HEAD ?
Next
From: Avery Payne
Date:
Subject: Re: A Silly Idea for Vertically-Oriented Databases