Re: Improving REINDEX for system indexes (long)

From: Alvaro Herrera <alvherre(at)dcc(dot)uchile(dot)cl>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Hiroshi Inoue <inoue(at)tpf(dot)co(dot)jp>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Improving REINDEX for system indexes (long)
Date: 2003-09-28 00:10:21
Message-ID: 20030928001021.GA9141@dcc.uchile.cl
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers pgsql-patches

On Sat, Sep 27, 2003 at 06:37:22PM -0400, Tom Lane wrote:
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > Tom, would you summarize what REINDEX currently _doesn't_ do?
>
> As of CVS tip I think the only deficiency is that indexes on the shared
> catalogs (pg_database, pg_shadow, pg_group) have to be reindexed in
> place, rather than being rebuilt with a new relfilenode as is done for
> CLUSTER or TRUNCATE. In-place reindexing isn't crash-safe, since if
> you fail you're left with a half-built (effectively corrupt) index.
>
> I don't see any way to avoid that, though, since we cannot change the
> relfilenode value for a shared index.

What about creating a separate filenode anyway and renaming the files
afterwards? It would not be an atomic operation anyway, but it would be
better than the current setup IMHO.

(It seems unlikely that one would have > 1GB indexes for shared
relations...)

--
Alvaro Herrera (<alvherre[a]dcc.uchile.cl>)
"La espina, desde que nace, ya pincha" (Proverbio africano)

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2003-09-28 00:15:48 Re: Improving REINDEX for system indexes (long)
Previous Message Bruce Momjian 2003-09-27 23:51:16 Re: Improving REINDEX for system indexes (long)

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2003-09-28 00:15:48 Re: Improving REINDEX for system indexes (long)
Previous Message Bruce Momjian 2003-09-27 23:51:16 Re: Improving REINDEX for system indexes (long)

Browse pgsql-patches by date

  From Date Subject
Next Message Alvaro Herrera 2003-09-28 00:11:16 Re: [PATCHES] bug in clusterdb script
Previous Message Bruce Momjian 2003-09-28 00:02:17 Re: bug in clusterdb script