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