On Friday 16 July 2010 20:41:44 Andres Freund wrote:
> >> !                     */
> >> !                    case AT_AddColumn:                      /* may
> >> rewrite heap, in some cases and visible to SELECT */ !
> >>                    case AT_DropColumn:                     /* change
> >> visible to SELECT */ !                    case
> >> AT_AddColumnToView:        /* CREATE VIEW */ !                    case
> >> AT_AlterColumnType:        /* must rewrite heap */ !
> >>                    case AT_DropConstraint:         /* as DROP INDEX */
> >> !                    case AT_AddOids:
> >> !                    case AT_DropOids:                       /* calls
> >> AT_DropColumn */ !                    case
> >> AT_EnableAlwaysRule:       /* as DROP INDEX */ !
> >>                    case AT_EnableReplicaRule:      /* as DROP INDEX */
> >> !                    case AT_EnableRule:                     /* as DROP
> >> INDEX */
Another remark:

Imho it would be usefull to keep that list in same order as in the enum - 
currently its hard to make sure no case is missing.

Andres

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to