Skip site navigation (1) Skip section navigation (2)

Re: pgsql/src/backend/tcop utility.c

From: Hiroshi Inoue <Inoue(at)tpf(dot)co(dot)jp>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql/src/backend/tcop utility.c
Date: 2002-02-08 00:29:41
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgsql-committers
Tom Lane wrote:
> inoue(at)postgresql(dot)org writes:
> >       Removed a check for REINDEX TABLE.
> As I commented before, what is the point of forbidding REINDEX INDEX
> if you are going to allow REINDEX TABLE?  Or REINDEX DATABASE for that
> matter?  You may as well strip out the corresponding checks in the
> other two cases too.

Simply because I've never tested REINDEX INDEX/DATABASE case
under postmaster and I'm afraid of the following. 
1) I didn't make any device to not look up the reindexing
   index itself in case of REINDEX INDEX.
2) REINDEX DATABASE generates multiple transactions internally.
   There's no such command other than VACUUM.

Hiroshi Inoue

In response to

pgsql-committers by date

Next:From: momjianDate: 2002-02-08 04:32:20
Subject: pgsql/doc TODO
Previous:From: tglDate: 2002-02-07 22:20:46
Subject: pgsql/src/backend/postmaster pgstat.c

Privacy Policy | About PostgreSQL
Copyright © 1996-2018 The PostgreSQL Global Development Group