From b12e77ab0e2588cd6e3c0c1453de66168030a823 Mon Sep 17 00:00:00 2001
From: Jason Petersen <jason@citusdata.com>
Date: Mon, 27 Nov 2017 11:43:23 -0700
Subject: [PATCH] Ensure unsupported VACUUMs don't go to workers
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Apparently these two blocks have been incorrect for nearly a year…
---
 src/backend/distributed/executor/multi_utility.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/backend/distributed/executor/multi_utility.c b/src/backend/distributed/executor/multi_utility.c
index 0275c3d03..339e6e073 100644
--- a/src/backend/distributed/executor/multi_utility.c
+++ b/src/backend/distributed/executor/multi_utility.c
@@ -1557,6 +1557,8 @@ IsSupportedDistributedVacuumStmt(Oid relationId, VacuumStmt *vacuumStmt)
 						  errhint("Set citus.enable_ddl_propagation to true in order to "
 								  "send targeted %s commands to worker nodes.",
 								  stmtName)));
+
+		return false;
 	}
 
 	if (vacuumStmt->options & VACOPT_VERBOSE)
@@ -1564,6 +1566,8 @@ IsSupportedDistributedVacuumStmt(Oid relationId, VacuumStmt *vacuumStmt)
 		ereport(ERROR, (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
 						errmsg("the VERBOSE option is currently unsupported in "
 							   "distributed %s commands", stmtName)));
+
+		return false;
 	}
 
 	return true;