Re: pgsql-server/ oc/src/sgml/ref/cluster.sgml rc/ ...

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Christopher Kings-Lynne <chriskl(at)familyhealth(dot)com(dot)au>
Cc: Bruce Momjian - CVS <momjian(at)postgresql(dot)org>, pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql-server/ oc/src/sgml/ref/cluster.sgml rc/ ...
Date: 2002-11-15 03:23:20
Message-ID: 200211150323.gAF3NKX02316@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers


It is my understanding that this was an improved version that addressed
Tom's comments.

In fact, I see in his comments:

> > - The multi-relation version (CLUSTER ALL) uses a multitransaction
> > approach, similar to what VACUUM does.

so I think this is the improved version.

---------------------------------------------------------------------------

Christopher Kings-Lynne wrote:
> Hey Bruce,
>
> I'm not certain, but didn't Tom object to this patch as it made CLUSER a
> non-transactional command?
>
> Chris
>
> > -----Original Message-----
> > From: pgsql-committers-owner(at)postgresql(dot)org
> > [mailto:pgsql-committers-owner(at)postgresql(dot)org]On Behalf Of Bruce Momjian
> > - CVS
> > Sent: Friday, 15 November 2002 11:10 AM
> > To: pgsql-committers(at)postgresql(dot)org
> > Subject: [COMMITTERS] pgsql-server/ oc/src/sgml/ref/cluster.sgml rc/ ...
> >
> >
> > CVSROOT: /cvsroot
> > Module name: pgsql-server
> > Changes by: momjian(at)postgresql(dot)org 02/11/14 22:09:39
> >
> > Modified files:
> > doc/src/sgml/ref: cluster.sgml
> > src/backend/commands: cluster.c
> > src/backend/parser: gram.y
> > src/backend/tcop: utility.c
> > src/include/commands: cluster.h
> > src/include/nodes: parsenodes.h
> > src/test/regress/expected: cluster.out
> > src/test/regress/sql: cluster.sql
> >
> > Log message:
> > New version attached. The following is implemented:
> >
> > - CLUSTER ALL clusters all the tables that have some index with
> > indisclustered set and the calling user owns.
> > - CLUSTER tablename clusters the named table, using the index with
> > indisclustered set. If no index has the bit set, throws
> > elog(ERROR).
> > - The multi-relation version (CLUSTER ALL) uses a multitransaction
> > approach, similar to what VACUUM does.
> >
> > Alvaro Herrera
> >
> >
> > ---------------------------(end of broadcast)---------------------------
> > TIP 6: Have you searched our list archives?
> >
> > http://archives.postgresql.org
> >
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 4: Don't 'kill -9' the postmaster
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2002-11-15 03:25:32 Re: pgsql-server/ oc/src/sgml/ref/cluster.sgml rc/ ...
Previous Message Bruce Momjian - CVS 2002-11-15 03:22:30 pgsql-server/doc/src/sgml plpgsql.sgml runtime ...