Move ALTER ... ALL IN to ProcessUtilitySlow

Now that ALTER TABLE .. ALL IN TABLESPACE has replaced the previous
ALTER TABLESPACE approach, it makes sense to move the calls down in
to ProcessUtilitySlow where the rest of ALTER TABLE is handled.

This also means that event triggers will support ALTER TABLE .. ALL
(which was the impetus for the original change, though it has other
good qualities also).

Álvaro Herrera

Back-patch to 9.4 as the original rework was.

diff --git a/src/backend/tcop/utility.c b/src/backend/tcop/utility.c
index 18c74525fa02fc940d6529df5adaa59573545728..d55d095585890d7326120d3c0bc66307da15007d 100644 (file)
--- a/src/backend/tcop/utility.c
+++ b/src/backend/tcop/utility.c
@@ -506,10 +506,6 @@ standard_ProcessUtility(Node *parsetree,
                        AlterTableSpaceOptions((AlterTableSpaceOptionsStmt *) parsetree);
                        break;
 
-               case T_AlterTableMoveAllStmt:
-                       AlterTableMoveAll((AlterTableMoveAllStmt *) parsetree);
-                       break;
-
                case T_TruncateStmt:
                        ExecuteTruncate((TruncateStmt *) parsetree);
                        break;
@@ -1291,6 +1287,10 @@ ProcessUtilitySlow(Node *parsetree,
                                AlterTSConfiguration((AlterTSConfigurationStmt *) parsetree);
                                break;
 
+                       case T_AlterTableMoveAllStmt:
+                               AlterTableMoveAll((AlterTableMoveAllStmt *) parsetree);
+                               break;
+
                        case T_DropStmt:
                                ExecDropStmt((DropStmt *) parsetree, isTopLevel);
                                break;