From 106e4b29c0b1fdba36193bf38896d77fc28ea940 Mon Sep 17 00:00:00 2001 From: Stephen Frost Date: Tue, 9 Sep 2014 10:52:10 -0400 Subject: [PATCH] Move ALTER ... ALL IN to ProcessUtilitySlow MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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. --- src/backend/tcop/utility.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/backend/tcop/utility.c b/src/backend/tcop/utility.c index 18c74525fa..d55d095585 100644 --- 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;