pg_query 2.2.0 → 4.2.0

data/ext/pg_query/pg_query_deparse.c

@@ -204,7 +204,7 @@ static void deparsePreparableStmt(StringInfo str, Node *node);
 static void deparseRuleActionStmt(StringInfo str, Node *node);
 static void deparseExplainableStmt(StringInfo str, Node *node);
 static void deparseStmt(StringInfo str, Node *node);
-static void deparseValue(StringInfo str, Value *value, DeparseNodeContext context);
+static void deparseValue(StringInfo str, union ValUnion *value, DeparseNodeContext context);
 
 // "any_name" in gram.y
 static void deparseAnyName(StringInfo str, List *parts)
@@ -249,6 +249,8 @@ static void deparseAnyNameSkipLast(StringInfo str, List *parts)
 // "a_expr" / "b_expr" in gram.y
 static void deparseExpr(StringInfo str, Node *node)
 {
+	if (node == NULL)
+		return;
 	switch (nodeTag(node))
 	{
 		case T_FuncCall:
@@ -381,7 +383,6 @@ static void deparseCExpr(StringInfo str, Node *node)
 static void deparseExprList(StringInfo str, List *exprs)
 {
 	ListCell *lc;
-
 	foreach(lc, exprs)
 	{
 		deparseExpr(str, lfirst(lc));
@@ -460,15 +461,15 @@ static void deparseSimpleTypename(StringInfo str, Node *node)
 }
 
 // "NumericOnly" in gram.y
-static void deparseNumericOnly(StringInfo str, Value *value)
+static void deparseNumericOnly(StringInfo str, union ValUnion *value)
 {
 	switch (nodeTag(value))
 	{
 		case T_Integer:
-			appendStringInfo(str, "%d", value->val.ival);
+			appendStringInfo(str, "%d", value->ival.ival);
 			break;
 		case T_Float:
-			appendStringInfoString(str, value->val.str);
+			appendStringInfoString(str, value->sval.sval);
 			break;
 		default:
 			Assert(false);
@@ -482,7 +483,7 @@ static void deparseNumericOnlyList(StringInfo str, List *l)
 
 	foreach(lc, l)
 	{
-		deparseNumericOnly(str, (Value *) lfirst(lc));
+		deparseNumericOnly(str, (union ValUnion *) lfirst(lc));
 		if (lnext(l, lc))
 			appendStringInfoString(str, ", ");
 	}
@@ -501,25 +502,25 @@ static void deparseSeqOptElem(StringInfo str, DefElem *def_elem)
 	else if (strcmp(def_elem->defname, "cache") == 0)
 	{
 		appendStringInfoString(str, "CACHE ");
-		deparseNumericOnly(str, (Value *) def_elem->arg);
+		deparseNumericOnly(str, (union ValUnion *) def_elem->arg);
 	}
-	else if (strcmp(def_elem->defname, "cycle") == 0 && intVal(def_elem->arg) == 1)
+	else if (strcmp(def_elem->defname, "cycle") == 0 && boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "CYCLE");
 	}
-	else if (strcmp(def_elem->defname, "cycle") == 0 && intVal(def_elem->arg) == 0)
+	else if (strcmp(def_elem->defname, "cycle") == 0 && !boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "NO CYCLE");
 	}
 	else if (strcmp(def_elem->defname, "increment") == 0)
 	{
 		appendStringInfoString(str, "INCREMENT ");
-		deparseNumericOnly(str, (Value *) def_elem->arg);
+		deparseNumericOnly(str, (union ValUnion *) def_elem->arg);
 	}
 	else if (strcmp(def_elem->defname, "maxvalue") == 0 && def_elem->arg != NULL)
 	{
 		appendStringInfoString(str, "MAXVALUE ");
-		deparseNumericOnly(str, (Value *) def_elem->arg);
+		deparseNumericOnly(str, (union ValUnion *) def_elem->arg);
 	}
 	else if (strcmp(def_elem->defname, "maxvalue") == 0 && def_elem->arg == NULL)
 	{
@@ -528,7 +529,7 @@ static void deparseSeqOptElem(StringInfo str, DefElem *def_elem)
 	else if (strcmp(def_elem->defname, "minvalue") == 0 && def_elem->arg != NULL)
 	{
 		appendStringInfoString(str, "MINVALUE ");
-		deparseNumericOnly(str, (Value *) def_elem->arg);
+		deparseNumericOnly(str, (union ValUnion *) def_elem->arg);
 	}
 	else if (strcmp(def_elem->defname, "minvalue") == 0 && def_elem->arg == NULL)
 	{
@@ -547,7 +548,7 @@ static void deparseSeqOptElem(StringInfo str, DefElem *def_elem)
 	else if (strcmp(def_elem->defname, "start") == 0)
 	{
 		appendStringInfoString(str, "START ");
-		deparseNumericOnly(str, (Value *) def_elem->arg);
+		deparseNumericOnly(str, (union ValUnion *) def_elem->arg);
 	}
 	else if (strcmp(def_elem->defname, "restart") == 0 && def_elem->arg == NULL)
 	{
@@ -556,7 +557,7 @@ static void deparseSeqOptElem(StringInfo str, DefElem *def_elem)
 	else if (strcmp(def_elem->defname, "restart") == 0 && def_elem->arg != NULL)
 	{
 		appendStringInfoString(str, "RESTART ");
-		deparseNumericOnly(str, (Value *) def_elem->arg);
+		deparseNumericOnly(str, (union ValUnion *) def_elem->arg);
 	}
 	else
 	{
@@ -711,7 +712,7 @@ static void deparseDefArg(StringInfo str, Node *arg, bool is_operator_def_arg)
 	}
 	else if (IsA(arg, Float) || IsA(arg, Integer)) // NumericOnly
 	{
-		deparseValue(str, (Value *) arg, DEPARSE_NODE_CONTEXT_NONE);
+		deparseValue(str, (union ValUnion *) arg, DEPARSE_NODE_CONTEXT_NONE);
 	}
 	else if (IsA(arg, String))
 	{
@@ -792,11 +793,11 @@ static void deparseCreateGenericOptions(StringInfo str, List *options)
 // "common_func_opt_item" in gram.y
 static void deparseCommonFuncOptItem(StringInfo str, DefElem *def_elem)
 {
-	if (strcmp(def_elem->defname, "strict") == 0 && intVal(def_elem->arg) == 1)
+	if (strcmp(def_elem->defname, "strict") == 0 && boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "RETURNS NULL ON NULL INPUT");
 	}
-	else if (strcmp(def_elem->defname, "strict") == 0 && intVal(def_elem->arg) == 0)
+	else if (strcmp(def_elem->defname, "strict") == 0 && !boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "CALLED ON NULL INPUT");
 	}
@@ -812,31 +813,31 @@ static void deparseCommonFuncOptItem(StringInfo str, DefElem *def_elem)
 	{
 		appendStringInfoString(str, "VOLATILE");
 	}
-	else if (strcmp(def_elem->defname, "security") == 0 && intVal(def_elem->arg) == 1)
+	else if (strcmp(def_elem->defname, "security") == 0 && boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "SECURITY DEFINER");
 	}
-	else if (strcmp(def_elem->defname, "security") == 0 && intVal(def_elem->arg) == 0)
+	else if (strcmp(def_elem->defname, "security") == 0 && !boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "SECURITY INVOKER");
 	}
-	else if (strcmp(def_elem->defname, "leakproof") == 0 && intVal(def_elem->arg) == 1)
+	else if (strcmp(def_elem->defname, "leakproof") == 0 && boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "LEAKPROOF");
 	}
-	else if (strcmp(def_elem->defname, "leakproof") == 0 && intVal(def_elem->arg) == 0)
+	else if (strcmp(def_elem->defname, "leakproof") == 0 && !boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "NOT LEAKPROOF");
 	}
 	else if (strcmp(def_elem->defname, "cost") == 0)
 	{
 		appendStringInfoString(str, "COST ");
-		deparseValue(str, (Value *) def_elem->arg, DEPARSE_NODE_CONTEXT_NONE);
+		deparseValue(str, (union ValUnion *) def_elem->arg, DEPARSE_NODE_CONTEXT_NONE);
 	}
 	else if (strcmp(def_elem->defname, "rows") == 0)
 	{
 		appendStringInfoString(str, "ROWS ");
-		deparseValue(str, (Value *) def_elem->arg, DEPARSE_NODE_CONTEXT_NONE);
+		deparseValue(str, (union ValUnion *) def_elem->arg, DEPARSE_NODE_CONTEXT_NONE);
 	}
 	else if (strcmp(def_elem->defname, "support") == 0)
 	{
@@ -993,19 +994,22 @@ static void deparseFuncName(StringInfo str, List *func_name)
 static void deparseFunctionWithArgtypes(StringInfo str, ObjectWithArgs *object_with_args)
 {
 	ListCell *lc;
-
 	deparseFuncName(str, object_with_args->objname);
 
 	if (!object_with_args->args_unspecified)
 	{
 		appendStringInfoChar(str, '(');
-		foreach(lc, object_with_args->objargs)
+		List *objargs = object_with_args->objargs;
+		if (object_with_args->objfuncargs)
+			objargs = object_with_args->objfuncargs;
+
+		foreach(lc, objargs)
 		{
-			if (IsA(lfirst(lc), TypeName))
-				deparseTypeName(str, castNode(TypeName, lfirst(lc)));
-			else
+			if (IsA(lfirst(lc), FunctionParameter))
 				deparseFunctionParameter(str, castNode(FunctionParameter, lfirst(lc)));
-			if (lnext(object_with_args->objargs, lc))
+			else
+				deparseTypeName(str, castNode(TypeName, lfirst(lc)));
+			if (lnext(objargs, lc))
 				appendStringInfoString(str, ", ");
 		}
 		appendStringInfoChar(str, ')');
@@ -1095,16 +1099,23 @@ static void deparseAggregateWithArgtypes(StringInfo str, ObjectWithArgs *object_
 	deparseFuncName(str, object_with_args->objname);
 
 	appendStringInfoChar(str, '(');
-	if (object_with_args->objargs == NULL)
+	if (object_with_args->objargs == NULL && object_with_args->objfuncargs == NULL)
 	{
 		appendStringInfoChar(str, '*');
 	}
 	else
 	{
-		foreach(lc, object_with_args->objargs)
+		List *objargs = object_with_args->objargs;
+		if (object_with_args->objfuncargs)
+			objargs = object_with_args->objfuncargs;
+
+		foreach(lc, objargs)
 		{
-			deparseTypeName(str, castNode(TypeName, lfirst(lc)));
-			if (lnext(object_with_args->objargs, lc))
+			if (IsA(lfirst(lc), FunctionParameter))
+				deparseFunctionParameter(str, castNode(FunctionParameter, lfirst(lc)));
+			else
+				deparseTypeName(str, castNode(TypeName, lfirst(lc)));
+			if (lnext(objargs, lc))
 				appendStringInfoString(str, ", ");
 		}
 	}
@@ -1235,6 +1246,64 @@ static void deparseOptBooleanOrString(StringInfo str, char *s)
 		deparseNonReservedWordOrSconst(str, s);
 }
 
+static void deparseOptBoolean(StringInfo str, Node *node)
+{
+	if (node == NULL)
+	{
+		return;
+	}
+
+	switch (nodeTag(node))
+	{
+		case T_String:
+			appendStringInfo(str, " %s", strVal(node));
+			break;
+		case T_Integer:
+			appendStringInfo(str, " %d", intVal(node));
+			break;
+		case T_Boolean:
+			appendStringInfo(str, " %s", boolVal(node) ? "TRUE" : "FALSE");
+			break;
+		default:
+			Assert(false);
+			break;
+	}
+}
+
+bool optBooleanValue(Node *node)
+{
+	if (node == NULL)
+	{
+		return true;
+	}
+
+	switch (nodeTag(node))
+	{
+		case T_String: {
+			// Longest valid string is "off\0"
+			char lower[4];
+			strncpy(lower, strVal(node), 4);
+			lower[3] = 0;
+
+			if (strcmp(lower, "on") == 0) {
+				return true;
+			} else if (strcmp(lower, "off") == 0) {
+				return false;
+			}
+
+			// No sane way to handle this.
+			return false;
+		}
+		case T_Integer:
+			return intVal(node) != 0;
+		case T_Boolean:
+			return boolVal(node);
+		default:
+			Assert(false);
+			return false;
+	}
+}
+
 // "var_name"
 //
 // Note this is kept separate from ColId in case we want to improve the
@@ -1259,7 +1328,7 @@ static void deparseVarList(StringInfo str, List *l)
 		{
 			A_Const *a_const = castNode(A_Const, lfirst(lc));
 			if (IsA(&a_const->val, Integer) || IsA(&a_const->val, Float))
-				deparseNumericOnly(str, (Value *) &a_const->val);
+				deparseNumericOnly(str, (union ValUnion *) &a_const->val);
 			else if (IsA(&a_const->val, String))
 				deparseOptBooleanOrString(str, strVal(&a_const->val));
 			else
@@ -1340,7 +1409,7 @@ static void deparseAlterIdentityColumnOptionList(StringInfo str, List *l)
 		else if (strcmp(def_elem->defname, "restart") == 0 && def_elem->arg != NULL)
 		{
 			appendStringInfoString(str, "RESTART ");
-			deparseNumericOnly(str, (Value *) def_elem->arg);
+			deparseNumericOnly(str, (union ValUnion *) def_elem->arg);
 		}
 		else if (strcmp(def_elem->defname, "generated") == 0)
 		{
@@ -2010,6 +2079,38 @@ static void deparseCreatedbOptList(StringInfo str, List *l)
 	}
 }
 
+// "utility_option_list" in gram.y
+static void deparseUtilityOptionList(StringInfo str, List *options)
+{
+	ListCell *lc = NULL;
+	char *defname = NULL;
+
+	if (list_length(options) > 0)
+	{
+		appendStringInfoChar(str, '(');
+		foreach(lc, options)
+		{
+			DefElem *def_elem = castNode(DefElem, lfirst(lc));
+			deparseGenericDefElemName(str, def_elem->defname);
+
+			if (def_elem->arg != NULL)
+			{
+				appendStringInfoChar(str, ' ');
+				if (IsA(def_elem->arg, Integer) || IsA(def_elem->arg, Float))
+					deparseNumericOnly(str, (union ValUnion *) def_elem->arg);
+				else if (IsA(def_elem->arg, String))
+					deparseOptBooleanOrString(str, strVal(def_elem->arg));
+				else
+					Assert(false);
+			}
+
+			if (lnext(options, lc))
+				appendStringInfoString(str, ", ");
+		}
+		appendStringInfoString(str, ") ");
+	}
+}
+
 static void deparseSelectStmt(StringInfo str, SelectStmt *stmt)
 {
 	const ListCell *lc = NULL;
@@ -2074,6 +2175,8 @@ static void deparseSelectStmt(StringInfo str, SelectStmt *stmt)
 			if (list_length(stmt->groupClause) > 0)
 			{
 				appendStringInfoString(str, "GROUP BY ");
+				if (stmt->groupDistinct)
+					appendStringInfoString(str, "DISTINCT ");
 				deparseGroupByList(str, stmt->groupClause);
 				appendStringInfoChar(str, ' ');
 			}
@@ -2159,7 +2262,7 @@ static void deparseSelectStmt(StringInfo str, SelectStmt *stmt)
 		else if (stmt->limitOption == LIMIT_OPTION_WITH_TIES)
 			appendStringInfoString(str, "FETCH FIRST ");
 
-		if (IsA(stmt->limitCount, A_Const) && IsA(&castNode(A_Const, stmt->limitCount)->val, Null))
+		if (IsA(stmt->limitCount, A_Const) && castNode(A_Const, stmt->limitCount)->isnull)
 			appendStringInfoString(str, "ALL");
 		else
 			deparseCExpr(str, stmt->limitCount);
@@ -2295,7 +2398,8 @@ static void deparseAlias(StringInfo str, Alias *alias)
 
 static void deparseAConst(StringInfo str, A_Const *a_const)
 {
-	deparseValue(str, &a_const->val, DEPARSE_NODE_CONTEXT_CONSTANT);
+	union ValUnion *val = a_const->isnull ? NULL : &a_const->val;
+	deparseValue(str, val, DEPARSE_NODE_CONTEXT_CONSTANT);
 }
 
 static void deparseFuncCall(StringInfo str, FuncCall *func_call)
@@ -2323,8 +2427,217 @@ static void deparseFuncCall(StringInfo str, FuncCall *func_call)
 		deparseExpr(str, lfourth(func_call->args));
 		appendStringInfoChar(str, ')');
 		return;
-	}
+	} else if (func_call->funcformat == COERCE_SQL_SYNTAX &&
+		list_length(func_call->funcname) == 2 &&
+		strcmp(strVal(linitial(func_call->funcname)), "pg_catalog") == 0 &&
+		strcmp(strVal(lsecond(func_call->funcname)), "substring") == 0)
+	{
+		/*
+		 * "SUBSTRING" is a keyword on its own merit, and only accepts the
+		 * keyword parameter style when its called as a keyword, not as a regular function (i.e. pg_catalog.substring)
+		 */
+		Assert(list_length(func_call->args) == 2 || list_length(func_call->args) == 3);
+		appendStringInfoString(str, "SUBSTRING(");
+		deparseExpr(str, linitial(func_call->args));
+		appendStringInfoString(str, " FROM ");
+		deparseExpr(str, lsecond(func_call->args));
+		if (list_length(func_call->args) == 3)
+		{
+			appendStringInfoString(str, " FOR ");
+			deparseExpr(str, lthird(func_call->args));
+		}
+		appendStringInfoChar(str, ')');
+		return;
+	} else if (func_call->funcformat == COERCE_SQL_SYNTAX &&
+		list_length(func_call->funcname) == 2 &&
+		strcmp(strVal(linitial(func_call->funcname)), "pg_catalog") == 0 &&
+		strcmp(strVal(lsecond(func_call->funcname)), "position") == 0 &&
+		list_length(func_call->args) == 2)
+	{
+		/*
+		 * "POSITION" is a keyword on its own merit, and only accepts the
+		 * keyword parameter style when its called as a keyword, not as a regular function (i.e. pg_catalog.position)
+		 * Note that the first and second arguments are switched in this format
+		 */
+		appendStringInfoString(str, "POSITION(");
+		deparseExpr(str, lsecond(func_call->args));
+		appendStringInfoString(str, " IN ");
+		deparseExpr(str, linitial(func_call->args));
+		appendStringInfoChar(str, ')');
+		return;
+	} else if (func_call->funcformat == COERCE_SQL_SYNTAX &&
+		list_length(func_call->funcname) == 2 &&
+		strcmp(strVal(linitial(func_call->funcname)), "pg_catalog") == 0 &&
+		strcmp(strVal(lsecond(func_call->funcname)), "overlay") == 0 &&
+		list_length(func_call->args) == 3)
+	{
+		/*
+		 * "OVERLAY" is a keyword on its own merit, and only accepts the
+		 * keyword parameter style when its called as a keyword, not as a regular function (i.e. pg_catalog.overlay)
+		 */
+		appendStringInfoString(str, "overlay(");
+		deparseExpr(str, linitial(func_call->args));
+		appendStringInfoString(str, " placing ");
+		deparseExpr(str, lsecond(func_call->args));
+		appendStringInfoString(str, " from ");
+		deparseExpr(str, lthird(func_call->args));
+		appendStringInfoChar(str, ')');
+		return;
+	} else if (func_call->funcformat == COERCE_SQL_SYNTAX &&
+		list_length(func_call->funcname) == 2 &&
+		strcmp(strVal(linitial(func_call->funcname)), "pg_catalog") == 0 &&
+		strcmp(strVal(lsecond(func_call->funcname)), "pg_collation_for") == 0 &&
+		list_length(func_call->args) == 1)
+	{
+		/*
+		 * "collation for" is a keyword on its own merit, and only accepts the
+		 * keyword parameter style when its called as a keyword, not as a regular function (i.e. pg_catalog.overlay)
+		 */
+		appendStringInfoString(str, "collation for (");
+		deparseExpr(str, linitial(func_call->args));
+		appendStringInfoChar(str, ')');
+		return;
+	} else if (func_call->funcformat == COERCE_SQL_SYNTAX &&
+		list_length(func_call->funcname) == 2 &&
+		strcmp(strVal(linitial(func_call->funcname)), "pg_catalog") == 0 &&
+		strcmp(strVal(lsecond(func_call->funcname)), "extract") == 0 &&
+		list_length(func_call->args) == 2)
+	{
+		/*
+		 * "EXTRACT" is a keyword on its own merit, and only accepts the
+		 * keyword parameter style when its called as a keyword, not as a regular function (i.e. pg_catalog.extract)
+		 */
+		appendStringInfoString(str, "extract (");
+		deparseExpr(str, linitial(func_call->args));
+		appendStringInfoString(str, " FROM ");
+		deparseExpr(str, lsecond(func_call->args));
+		appendStringInfoChar(str, ')');
+		return;
+	} else if (func_call->funcformat == COERCE_SQL_SYNTAX &&
+		list_length(func_call->funcname) == 2 &&
+		strcmp(strVal(linitial(func_call->funcname)), "pg_catalog") == 0 &&
+		strcmp(strVal(lsecond(func_call->funcname)), "overlaps") == 0 &&
+		list_length(func_call->args) == 4)
+	{
+		/*
+		 * "OVERLAPS" is a keyword on its own merit, and only accepts the
+		 * keyword parameter style when its called as a keyword, not as a regular function (i.e. pg_catalog.overlaps)
+		 * format: (start_1, end_1) overlaps (start_2, end_2)
+		 */
+		appendStringInfoChar(str, '(');
+		deparseExpr(str, linitial(func_call->args));
+		appendStringInfoString(str, ", ");
+		deparseExpr(str, lsecond(func_call->args));
+		appendStringInfoString(str, ") ");
+
+		appendStringInfoString(str, "overlaps ");
+		appendStringInfoChar(str, '(');
+		deparseExpr(str, lthird(func_call->args));
+		appendStringInfoString(str, ", ");
+		deparseExpr(str, lfourth(func_call->args));
+		appendStringInfoString(str, ") ");
+		return;
+	} else if (func_call->funcformat == COERCE_SQL_SYNTAX &&
+		list_length(func_call->funcname) == 2 &&
+		strcmp(strVal(linitial(func_call->funcname)), "pg_catalog") == 0 &&
+		(
+			strcmp(strVal(lsecond(func_call->funcname)), "ltrim") == 0 ||
+			strcmp(strVal(lsecond(func_call->funcname)), "btrim") == 0 ||
+			strcmp(strVal(lsecond(func_call->funcname)), "rtrim") == 0
+		))
+	{
+		/*
+		 * "TRIM " is a keyword on its own merit, and only accepts the
+		 * keyword parameter style when its called as a keyword, not as a regular function (i.e. pg_catalog.ltrim)
+		 * Note that the first and second arguments are switched in this format
+		 */
+		Assert(list_length(func_call->args) == 1 || list_length(func_call->args) == 2);
+		appendStringInfoString(str, "TRIM (");
+		if (strcmp(strVal(lsecond(func_call->funcname)), "ltrim") == 0)
+			appendStringInfoString(str, "LEADING ");
+		else if (strcmp(strVal(lsecond(func_call->funcname)), "btrim") == 0)
+			appendStringInfoString(str, "BOTH ");
+		else if (strcmp(strVal(lsecond(func_call->funcname)), "rtrim") == 0)
+			appendStringInfoString(str, "TRAILING ");
+
+		if (list_length(func_call->args) == 2)
+			deparseExpr(str, lsecond(func_call->args));
+		appendStringInfoString(str, " FROM ");
+		deparseExpr(str, linitial(func_call->args));
+		appendStringInfoChar(str, ')');
+		return;
+	} else if (func_call->funcformat == COERCE_SQL_SYNTAX &&
+		list_length(func_call->funcname) == 2 &&
+		strcmp(strVal(linitial(func_call->funcname)), "pg_catalog") == 0 &&
+		strcmp(strVal(lsecond(func_call->funcname)), "timezone") == 0 &&
+		list_length(func_call->args) == 2)
+	{
+		/*
+		 * "AT TIME ZONE" is a keyword on its own merit, and only accepts the
+		 * keyword parameter style when its called as a keyword, not as a regular function (i.e. pg_catalog.timezone)
+		 * Note that the arguments are swapped in this case
+		 */
+		deparseExpr(str, lsecond(func_call->args));
+		appendStringInfoString(str, " AT TIME ZONE ");
+		deparseExpr(str, linitial(func_call->args));
+		return;
+	} else if (func_call->funcformat == COERCE_SQL_SYNTAX &&
+		list_length(func_call->funcname) == 2 &&
+		strcmp(strVal(linitial(func_call->funcname)), "pg_catalog") == 0 &&
+		strcmp(strVal(lsecond(func_call->funcname)), "normalize") == 0)
+	{
+		/*
+		 * "NORMALIZE" is a keyword on its own merit, and only accepts the
+		 * keyword parameter style when its called as a keyword, not as a regular function (i.e. pg_catalog.normalize)
+		 */
+		Assert(list_length(func_call->args) == 1 || list_length(func_call->args) == 2);
+		appendStringInfoString(str, "normalize (");
 
+		deparseExpr(str, linitial(func_call->args));
+		if (list_length(func_call->args) == 2)
+		{
+			appendStringInfoString(str, ", ");
+			Assert(IsA(lsecond(func_call->args), A_Const));
+			A_Const *aconst = lsecond(func_call->args);
+			deparseValue(str, &aconst->val, DEPARSE_NODE_CONTEXT_NONE);
+		}
+		appendStringInfoChar(str, ')');
+		return;
+	} else if (func_call->funcformat == COERCE_SQL_SYNTAX &&
+		list_length(func_call->funcname) == 2 &&
+		strcmp(strVal(linitial(func_call->funcname)), "pg_catalog") == 0 &&
+		strcmp(strVal(lsecond(func_call->funcname)), "is_normalized") == 0)
+	{
+		/*
+		 * "IS NORMALIZED" is a keyword on its own merit, and only accepts the
+		 * keyword parameter style when its called as a keyword, not as a regular function (i.e. pg_catalog.is_normalized)
+		 */
+		Assert(list_length(func_call->args) == 1 || list_length(func_call->args) == 2);
+
+		deparseExpr(str, linitial(func_call->args));
+		appendStringInfoString(str, " IS ");
+		if (list_length(func_call->args) == 2)
+		{
+			Assert(IsA(lsecond(func_call->args), A_Const));
+			A_Const *aconst = lsecond(func_call->args);
+			deparseValue(str, &aconst->val, DEPARSE_NODE_CONTEXT_NONE);
+		}
+		appendStringInfoString(str, " NORMALIZED ");
+		return;
+	} else if (func_call->funcformat == COERCE_SQL_SYNTAX &&
+		list_length(func_call->funcname) == 2 &&
+		strcmp(strVal(linitial(func_call->funcname)), "pg_catalog") == 0 &&
+		strcmp(strVal(lsecond(func_call->funcname)), "xmlexists") == 0 &&
+		list_length(func_call->args) == 2)
+	{
+		appendStringInfoString(str, "xmlexists (");
+		deparseExpr(str, linitial(func_call->args));
+		appendStringInfoString(str, " PASSING ");
+		deparseExpr(str, lsecond(func_call->args));
+		appendStringInfoChar(str, ')');
+		return;
+	}
+		
 	deparseFuncName(str, func_call->funcname);
 	appendStringInfoChar(str, '(');
 
@@ -2648,31 +2961,13 @@ static void deparseAExpr(StringInfo str, A_Expr* a_expr, DeparseNodeContext cont
 			deparseExpr(str, a_expr->rexpr);
 			appendStringInfoChar(str, ')');
 			return;
-		case AEXPR_OF: /* IS [NOT] OF - name must be "=" or "<>" */
-			Assert(list_length(a_expr->name) == 1);
-			Assert(IsA(linitial(a_expr->name), String));
-			Assert(IsA(a_expr->rexpr, List));
-			deparseExpr(str, a_expr->lexpr);
-			appendStringInfoChar(str, ' ');
-			name = ((Value *) linitial(a_expr->name))->val.str;
-			if (strcmp(name, "=") == 0) {
-				appendStringInfoString(str, "IS OF ");
-			} else if (strcmp(name, "<>") == 0) {
-				appendStringInfoString(str, "IS NOT OF ");
-			} else {
-				Assert(false);
-			}
-			appendStringInfoChar(str, '(');
-			deparseTypeList(str, castNode(List, a_expr->rexpr));
-			appendStringInfoChar(str, ')');
-			return;
 		case AEXPR_IN: /* [NOT] IN - name must be "=" or "<>" */
 			Assert(list_length(a_expr->name) == 1);
 			Assert(IsA(linitial(a_expr->name), String));
 			Assert(IsA(a_expr->rexpr, List));
 			deparseExpr(str, a_expr->lexpr);
 			appendStringInfoChar(str, ' ');
-			name = ((Value *) linitial(a_expr->name))->val.str;
+			name = ((union ValUnion *) linitial(a_expr->name))->sval.sval;
 			if (strcmp(name, "=") == 0) {
 				appendStringInfoString(str, "IN ");
 			} else if (strcmp(name, "<>") == 0) {
@@ -2693,7 +2988,7 @@ static void deparseAExpr(StringInfo str, A_Expr* a_expr, DeparseNodeContext cont
 			deparseExpr(str, a_expr->lexpr);
 			appendStringInfoChar(str, ' ');
 
-			name = ((Value *) linitial(a_expr->name))->val.str;
+			name = ((union ValUnion *) linitial(a_expr->name))->sval.sval;
 			if (strcmp(name, "~~") == 0) {
 				appendStringInfoString(str, "LIKE ");
 			} else if (strcmp(name, "!~~") == 0) {
@@ -2710,7 +3005,7 @@ static void deparseAExpr(StringInfo str, A_Expr* a_expr, DeparseNodeContext cont
 			deparseExpr(str, a_expr->lexpr);
 			appendStringInfoChar(str, ' ');
 
-			name = ((Value *) linitial(a_expr->name))->val.str;
+			name = ((union ValUnion *) linitial(a_expr->name))->sval.sval;
 			if (strcmp(name, "~~*") == 0) {
 				appendStringInfoString(str, "ILIKE ");
 			} else if (strcmp(name, "!~~*") == 0) {
@@ -2727,7 +3022,7 @@ static void deparseAExpr(StringInfo str, A_Expr* a_expr, DeparseNodeContext cont
 			deparseExpr(str, a_expr->lexpr);
 			appendStringInfoChar(str, ' ');
 
-			name = ((Value *) linitial(a_expr->name))->val.str;
+			name = ((union ValUnion *) linitial(a_expr->name))->sval.sval;
 			if (strcmp(name, "~") == 0) {
 				appendStringInfoString(str, "SIMILAR TO ");
 			} else if (strcmp(name, "!~") == 0) {
@@ -2769,10 +3064,6 @@ static void deparseAExpr(StringInfo str, A_Expr* a_expr, DeparseNodeContext cont
 					appendStringInfoString(str, " AND ");
 			}
 			return;
-		case AEXPR_PAREN: /* nameless dummy node for parentheses */
-			// Not present in parse trees when operator_precedence_warning is turned off
-			Assert(false);
-			return;
 	}
 }
 
@@ -3032,6 +3323,12 @@ static void deparseJoinExpr(StringInfo str, JoinExpr *join_expr)
 		appendStringInfoString(str, "USING (");
 		deparseNameList(str, join_expr->usingClause);
 		appendStringInfoString(str, ") ");
+
+		if (join_expr->join_using_alias)
+		{
+			appendStringInfoString(str, "AS ");
+			appendStringInfoString(str, join_expr->join_using_alias->aliasname);
+		}
 	}
 
 	if (need_alias_parens)
@@ -3043,6 +3340,49 @@ static void deparseJoinExpr(StringInfo str, JoinExpr *join_expr)
 	removeTrailingSpace(str);
 }
 
+static void deparseCTESearchClause(StringInfo str, CTESearchClause *search_clause)
+{
+	appendStringInfoString(str, " SEARCH ");
+	if (search_clause->search_breadth_first)
+		appendStringInfoString(str, "BREADTH ");
+	else
+		appendStringInfoString(str, "DEPTH ");
+
+	appendStringInfoString(str, "FIRST BY ");
+
+	if (search_clause->search_col_list)
+		deparseColumnList(str, search_clause->search_col_list);
+
+	appendStringInfoString(str, " SET ");
+	appendStringInfoString(str, quote_identifier(search_clause->search_seq_column));
+}
+
+static void deparseCTECycleClause(StringInfo str, CTECycleClause *cycle_clause)
+{
+	appendStringInfoString(str, " CYCLE ");
+
+	if (cycle_clause->cycle_col_list)
+		deparseColumnList(str, cycle_clause->cycle_col_list);
+
+	appendStringInfoString(str, " SET ");
+	appendStringInfoString(str, quote_identifier(cycle_clause->cycle_mark_column));
+
+	if (cycle_clause->cycle_mark_value)
+	{
+		appendStringInfoString(str, " TO ");
+		deparseExpr(str, cycle_clause->cycle_mark_value);
+	}
+	
+	if (cycle_clause->cycle_mark_default)
+	{
+		appendStringInfoString(str, " DEFAULT ");
+		deparseExpr(str, cycle_clause->cycle_mark_default);
+	}
+	
+	appendStringInfoString(str, " USING ");
+	appendStringInfoString(str, quote_identifier(cycle_clause->cycle_path_column));
+}
+
 static void deparseCommonTableExpr(StringInfo str, CommonTableExpr *cte)
 {
 	deparseColId(str, cte->ctename);
@@ -3070,6 +3410,11 @@ static void deparseCommonTableExpr(StringInfo str, CommonTableExpr *cte)
 	appendStringInfoChar(str, '(');
 	deparsePreparableStmt(str, cte->ctequery);
 	appendStringInfoChar(str, ')');
+
+	if (cte->search_clause)
+		deparseCTESearchClause(str, cte->search_clause);
+	if (cte->cycle_clause)
+		deparseCTECycleClause(str, cte->cycle_clause);
 }
 
 static void deparseRangeSubselect(StringInfo str, RangeSubselect *range_subselect)
@@ -3174,6 +3519,7 @@ static void deparseRowExpr(StringInfo str, RowExpr *row_expr)
 		case COERCE_EXPLICIT_CALL:
 			appendStringInfoString(str, "ROW");
 			break;
+		case COERCE_SQL_SYNTAX:
 		case COERCE_EXPLICIT_CAST:
 			// Not present in raw parser output
 			Assert(false);
@@ -3243,8 +3589,18 @@ static void deparseTypeCast(StringInfo str, TypeCast *type_cast)
 		{
 			need_parens = true;
 		}
+
+		if (list_length(type_cast->typeName->names) == 1 &&
+			strcmp(strVal(linitial(type_cast->typeName->names)), "point") == 0 &&
+			a_const->location > type_cast->typeName->location)
+		{
+			appendStringInfoString(str, " point ");
+			deparseAConst(str, a_const);
+			return;
+		}
 	}
 
+
 	if (need_parens)
 		appendStringInfoChar(str, '(');
 	deparseExpr(str, type_cast->arg);
@@ -3602,7 +3958,7 @@ static void deparseColumnDef(StringInfo str, ColumnDef *column_def)
 
 	if (column_def->colname != NULL)
 	{
-		appendStringInfoString(str, column_def->colname);
+		appendStringInfoString(str, quote_identifier(column_def->colname));
 		appendStringInfoChar(str, ' ');
 	}
 
@@ -3639,6 +3995,22 @@ static void deparseColumnDef(StringInfo str, ColumnDef *column_def)
 	removeTrailingSpace(str);
 }
 
+static void deparseInsertOverride(StringInfo str, OverridingKind override)
+{
+	switch (override)
+	{
+		case OVERRIDING_NOT_SET:
+			// Do nothing
+			break;
+		case OVERRIDING_USER_VALUE:
+			appendStringInfoString(str, "OVERRIDING USER VALUE ");
+			break;
+		case OVERRIDING_SYSTEM_VALUE:
+			appendStringInfoString(str, "OVERRIDING SYSTEM VALUE ");
+			break;
+	}
+}
+
 static void deparseInsertStmt(StringInfo str, InsertStmt *insert_stmt)
 {
 	ListCell *lc;
@@ -3661,18 +4033,7 @@ static void deparseInsertStmt(StringInfo str, InsertStmt *insert_stmt)
 		appendStringInfoString(str, ") ");
 	}
 
-	switch (insert_stmt->override)
-	{
-		case OVERRIDING_NOT_SET:
-			// Do nothing
-			break;
-		case OVERRIDING_USER_VALUE:
-			appendStringInfoString(str, "OVERRIDING USER VALUE ");
-			break;
-		case OVERRIDING_SYSTEM_VALUE:
-			appendStringInfoString(str, "OVERRIDING SYSTEM VALUE ");
-			break;
-	}
+	deparseInsertOverride(str, insert_stmt->override);
 
 	if (insert_stmt->selectStmt != NULL)
 	{
@@ -3799,6 +4160,90 @@ static void deparseUpdateStmt(StringInfo str, UpdateStmt *update_stmt)
 	removeTrailingSpace(str);
 }
 
+static void deparseMergeStmt(StringInfo str, MergeStmt *merge_stmt)
+{
+	if (merge_stmt->withClause != NULL)
+	{
+		deparseWithClause(str, merge_stmt->withClause);
+		appendStringInfoChar(str, ' ');
+	}
+
+	appendStringInfoString(str, "MERGE INTO ");
+	deparseRangeVar(str, merge_stmt->relation, DEPARSE_NODE_CONTEXT_NONE);
+	appendStringInfoChar(str, ' ');
+
+	appendStringInfoString(str, "USING ");
+	deparseTableRef(str, merge_stmt->sourceRelation);
+	appendStringInfoChar(str, ' ');
+
+	appendStringInfoString(str, "ON ");
+	deparseExpr(str, merge_stmt->joinCondition);
+	appendStringInfoChar(str, ' ');
+
+	ListCell *lc, *lc2;
+	foreach (lc, merge_stmt->mergeWhenClauses)
+	{
+		MergeWhenClause *clause = castNode(MergeWhenClause, lfirst(lc));
+
+		appendStringInfoString(str, "WHEN ");
+
+		if (!clause->matched)
+		{
+			appendStringInfoString(str, "NOT ");
+		}
+
+		appendStringInfoString(str, "MATCHED ");
+
+		if (clause->condition)
+		{
+			appendStringInfoString(str, "AND ");
+			deparseExpr(str, clause->condition);
+			appendStringInfoChar(str, ' ');
+		}
+
+		appendStringInfoString(str, "THEN ");
+
+		switch (clause->commandType) {
+			case CMD_INSERT:
+				appendStringInfoString(str, "INSERT ");
+
+				if (clause->targetList) {
+					appendStringInfoChar(str, '(');
+					deparseInsertColumnList(str, clause->targetList);
+					appendStringInfoString(str, ") ");
+				}
+
+				deparseInsertOverride(str, clause->override);
+
+				if (clause->values) {
+					appendStringInfoString(str, "VALUES (");
+					deparseExprList(str, clause->values);
+					appendStringInfoString(str, ")");
+				} else {
+					appendStringInfoString(str, "DEFAULT VALUES ");
+				}
+
+				break;
+			case CMD_UPDATE:
+				appendStringInfoString(str, "UPDATE SET ");
+				deparseSetClauseList(str, clause->targetList);
+				break;
+			case CMD_DELETE:
+				appendStringInfoString(str, "DELETE");
+				break;
+			case CMD_NOTHING:
+				appendStringInfoString(str, "DO NOTHING");
+				break;
+			default:
+				elog(ERROR, "deparse: unpermitted command type in merge statement: %d", clause->commandType);
+				break;
+		}
+
+		if (lfirst(lc) != llast(merge_stmt->mergeWhenClauses))
+			appendStringInfoChar(str, ' ');
+	}
+}
+
 static void deparseDeleteStmt(StringInfo str, DeleteStmt *delete_stmt)
 {
 	if (delete_stmt->withClause != NULL)
@@ -3914,6 +4359,10 @@ static void deparseCreateCastStmt(StringInfo str, CreateCastStmt *create_cast_st
 		case COERCION_ASSIGNMENT:
 			appendStringInfoString(str, "AS ASSIGNMENT");
 			break;
+		case COERCION_PLPGSQL:
+			// Not present in raw parser output
+			Assert(false);
+			break;
 		case COERCION_EXPLICIT:
 			// Default
 			break;
@@ -4294,10 +4743,24 @@ static void deparseConstraint(StringInfo str, Constraint *constraint)
 			appendStringInfoString(str, "ON DELETE CASCADE ");
 			break;
 		case FKCONSTR_ACTION_SETNULL:
-			appendStringInfoString(str, "ON DELETE SET NULL ");
-			break;
 		case FKCONSTR_ACTION_SETDEFAULT:
-			appendStringInfoString(str, "ON DELETE SET DEFAULT ");
+			appendStringInfoString(str, "ON DELETE SET ");
+
+			switch (constraint->fk_del_action) {
+				case FKCONSTR_ACTION_SETDEFAULT: appendStringInfoString(str, "DEFAULT "); break;
+				case FKCONSTR_ACTION_SETNULL:    appendStringInfoString(str, "NULL "); break;
+			}
+
+			if (constraint->fk_del_set_cols) {
+				appendStringInfoString(str, "(");
+				ListCell *lc;
+				foreach (lc, constraint->fk_del_set_cols) {
+					appendStringInfoString(str, strVal(lfirst(lc)));
+					if (lfirst(lc) != llast(constraint->fk_del_set_cols))
+						appendStringInfoString(str, ", ");
+				}
+				appendStringInfoString(str, ")");
+			}
 			break;
 		default:
 			// Not specified
@@ -4332,6 +4795,12 @@ static void deparseConstraint(StringInfo str, Constraint *constraint)
 	removeTrailingSpace(str);
 }
 
+static void deparseReturnStmt(StringInfo str, ReturnStmt *return_stmt)
+{
+	appendStringInfoString(str, "RETURN ");
+	deparseExpr(str, return_stmt->returnval);
+}
+
 static void deparseCreateFunctionStmt(StringInfo str, CreateFunctionStmt *create_function_stmt)
 {
 	ListCell *lc;
@@ -4392,6 +4861,20 @@ static void deparseCreateFunctionStmt(StringInfo str, CreateFunctionStmt *create
 		appendStringInfoChar(str, ' ');
 	}
 
+	if (create_function_stmt->sql_body)
+	{
+		/* RETURN or BEGIN ... END
+		 */
+		if (IsA(create_function_stmt->sql_body, ReturnStmt))
+			deparseReturnStmt(str, castNode(ReturnStmt, create_function_stmt->sql_body));
+		else
+		{
+			appendStringInfoString(str, "BEGIN ATOMIC ");
+			deparseExprList(str, castNode(List, create_function_stmt->sql_body));
+			appendStringInfoString(str, "END ");
+		}
+	}
+
 	removeTrailingSpace(str);
 }
 
@@ -4400,7 +4883,7 @@ static void deparseFunctionParameter(StringInfo str, FunctionParameter *function
 	switch (function_parameter->mode)
 	{
 		case FUNC_PARAM_IN: /* input only */
-			// Default
+			appendStringInfoString(str, "IN ");
 			break;
 		case FUNC_PARAM_OUT: /* output only */
 			appendStringInfoString(str, "OUT ");
@@ -4415,6 +4898,9 @@ static void deparseFunctionParameter(StringInfo str, FunctionParameter *function
 			// No special annotation, the caller is expected to correctly put
 			// this into the RETURNS part of the CREATE FUNCTION statement
 			break;
+		case FUNC_PARAM_DEFAULT:
+			// Default
+			break;
 		default:
 			Assert(false);
 			break;
@@ -4446,7 +4932,6 @@ static void deparseCheckPointStmt(StringInfo str, CheckPointStmt *check_point_st
 static void deparseCreateSchemaStmt(StringInfo str, CreateSchemaStmt *create_schema_stmt)
 {
 	ListCell *lc;
-
 	appendStringInfoString(str, "CREATE SCHEMA ");
 
 	if (create_schema_stmt->if_not_exists)
@@ -4465,11 +4950,14 @@ static void deparseCreateSchemaStmt(StringInfo str, CreateSchemaStmt *create_sch
 		appendStringInfoChar(str, ' ');
 	}
 
-	foreach(lc, create_schema_stmt->schemaElts)
+	if (create_schema_stmt->schemaElts)
 	{
-		deparseSchemaStmt(str, lfirst(lc));
-		if (lnext(create_schema_stmt->schemaElts, lc))
-			appendStringInfoChar(str, ' ');
+		foreach(lc, create_schema_stmt->schemaElts)
+		{
+			deparseSchemaStmt(str, lfirst(lc));
+			if (lnext(create_schema_stmt->schemaElts, lc))
+				appendStringInfoChar(str, ' ');
+		}
 	}
 
 	removeTrailingSpace(str);
@@ -4523,6 +5011,9 @@ static void deparseRoleSpec(StringInfo str, RoleSpec *role_spec)
 			Assert(role_spec->rolename != NULL);
 			appendStringInfoString(str, quote_identifier(role_spec->rolename));
 			break;
+		case ROLESPEC_CURRENT_ROLE:
+			appendStringInfoString(str, "CURRENT_ROLE");
+			break;
 		case ROLESPEC_CURRENT_USER:
 			appendStringInfoString(str, "CURRENT_USER");
 			break;
@@ -4619,6 +5110,8 @@ static void deparsePartitionCmd(StringInfo str, PartitionCmd *partition_cmd)
 		appendStringInfoChar(str, ' ');
 		deparsePartitionBoundSpec(str, partition_cmd->bound);
 	}
+	if (partition_cmd->concurrent)
+		appendStringInfoString(str, " CONCURRENTLY ");
 }
 
 // "TableElement" in gram.y
@@ -4983,7 +5476,7 @@ static void deparseSecLabelStmt(StringInfo str, SecLabelStmt *sec_label_stmt)
 			break;
 		case OBJECT_LARGEOBJECT:
 			appendStringInfoString(str, "LARGE OBJECT ");
-			deparseValue(str, (Value *) sec_label_stmt->object, DEPARSE_NODE_CONTEXT_CONSTANT);
+			deparseValue(str, (union ValUnion *) sec_label_stmt->object, DEPARSE_NODE_CONTEXT_CONSTANT);
 			break;
 		case OBJECT_PROCEDURE:
 			appendStringInfoString(str, "PROCEDURE ");
@@ -5067,7 +5560,7 @@ static void deparseCreateTableAsStmt(StringInfo str, CreateTableAsStmt *create_t
 
 	deparseOptTemp(str, create_table_as_stmt->into->rel->relpersistence);
 
-	switch (create_table_as_stmt->relkind)
+	switch (create_table_as_stmt->objtype)
 	{
 		case OBJECT_TABLE:
 			appendStringInfoString(str, "TABLE ");
@@ -5340,7 +5833,7 @@ static void deparseDropStmt(StringInfo str, DropStmt *drop_stmt)
 			appendStringInfoChar(str, ' ');
 			break;
 		case OBJECT_LANGUAGE:
-			deparseStringLiteral(str, strVal(linitial(drop_stmt->objects)));
+			deparseNameList(str, drop_stmt->objects);
 			appendStringInfoChar(str, ' ');
 			break;
 		case OBJECT_TYPE:
@@ -5471,8 +5964,7 @@ static void deparseAlterObjectDependsStmt(StringInfo str, AlterObjectDependsStmt
 	if (alter_object_depends_stmt->remove)
 		appendStringInfoString(str, "NO ");
 
-	appendStringInfoString(str, "DEPENDS ON EXTENSION ");
-	deparseColId(str, strVal(alter_object_depends_stmt->extname));
+	appendStringInfo(str, "DEPENDS ON EXTENSION %s", alter_object_depends_stmt->extname->sval);
 }
 
 static void deparseAlterObjectSchemaStmt(StringInfo str, AlterObjectSchemaStmt *alter_object_schema_stmt)
@@ -5662,6 +6154,10 @@ static void deparseAlterTableCmd(StringInfo str, AlterTableCmd *alter_table_cmd,
 			appendStringInfoString(str, "ALTER COLUMN ");
 			options = "SET STORAGE";
 			break;
+		case AT_SetCompression: /* alter column set compression */
+			appendStringInfoString(str, "ALTER COLUMN ");
+			options = "SET COMPRESSION";
+			break;
 		case AT_DropColumn: /* drop column */
 			if (context == DEPARSE_NODE_CONTEXT_ALTER_TYPE)
 				appendStringInfoString(str, "DROP ATTRIBUTE ");
@@ -5709,6 +6205,7 @@ static void deparseAlterTableCmd(StringInfo str, AlterTableCmd *alter_table_cmd,
 			Assert(false);
 			break;
 		case AT_ReAddComment: /* internal to commands/tablecmds.c */
+		case AT_ReAddStatistics: /* internal to commands/tablecmds.c */
 			Assert(false);
 			break;
 		case AT_AlterColumnType: /* alter column type */
@@ -5747,6 +6244,9 @@ static void deparseAlterTableCmd(StringInfo str, AlterTableCmd *alter_table_cmd,
 		case AT_SetRelOptions: /* SET (...) -- AM specific parameters */
 			appendStringInfoString(str, "SET ");
 			break;
+		case AT_SetAccessMethod:
+			appendStringInfo(str, "SET ACCESS METHOD ");
+			break;
 		case AT_ResetRelOptions: /* RESET (...) -- AM specific parameters */
 			appendStringInfoString(str, "RESET ");
 			break;
@@ -5826,6 +6326,9 @@ static void deparseAlterTableCmd(StringInfo str, AlterTableCmd *alter_table_cmd,
 		case AT_DetachPartition: /* DETACH PARTITION */
 			appendStringInfoString(str, "DETACH PARTITION ");
 			break;
+		case AT_DetachPartitionFinalize: /* DETACH PARTITION FINALIZE */
+			appendStringInfoString(str, "DETACH PARTITION ");
+			break;
 		case AT_AddIdentity: /* ADD IDENTITY */
 			appendStringInfoString(str, "ALTER ");
 			options = "ADD";
@@ -5874,6 +6377,10 @@ static void deparseAlterTableCmd(StringInfo str, AlterTableCmd *alter_table_cmd,
 			deparsePartitionCmd(str, castNode(PartitionCmd, alter_table_cmd->def));
 			appendStringInfoChar(str, ' ');
 			break;
+		case AT_DetachPartitionFinalize:
+			deparsePartitionCmd(str, castNode(PartitionCmd, alter_table_cmd->def));
+			appendStringInfoString(str, "FINALIZE ");
+			break;
 		case AT_AddColumn:
 		case AT_AlterColumnType:
 			deparseColumnDef(str, castNode(ColumnDef, alter_table_cmd->def));
@@ -5901,6 +6408,13 @@ static void deparseAlterTableCmd(StringInfo str, AlterTableCmd *alter_table_cmd,
 			deparseColId(str, strVal(alter_table_cmd->def));
 			appendStringInfoChar(str, ' ');
 			break;
+		case AT_SetCompression:
+			if (strcmp(strVal(alter_table_cmd->def), "default") == 0)
+				appendStringInfoString(str, "DEFAULT");
+			else
+				deparseColId(str, strVal(alter_table_cmd->def));
+			appendStringInfoChar(str, ' ');
+			break;
 		case AT_AddIdentity:
 		case AT_AddConstraint:
 		case AT_AlterConstraint:
@@ -5939,14 +6453,9 @@ static void deparseAlterTableCmd(StringInfo str, AlterTableCmd *alter_table_cmd,
 	removeTrailingSpace(str);
 }
 
-static void deparseAlterTableStmt(StringInfo str, AlterTableStmt *alter_table_stmt)
+static DeparseNodeContext deparseAlterTableObjType(StringInfo str, ObjectType type)
 {
-	ListCell *lc;
-	DeparseNodeContext context = DEPARSE_NODE_CONTEXT_NONE;
-
-	appendStringInfoString(str, "ALTER ");
-
-	switch (alter_table_stmt->relkind)
+	switch (type)
 	{
 		case OBJECT_TABLE:
 			appendStringInfoString(str, "TABLE ");
@@ -5968,13 +6477,49 @@ static void deparseAlterTableStmt(StringInfo str, AlterTableStmt *alter_table_st
 			break;
 		case OBJECT_TYPE:
 			appendStringInfoString(str, "TYPE ");
-			context = DEPARSE_NODE_CONTEXT_ALTER_TYPE;
+			return DEPARSE_NODE_CONTEXT_ALTER_TYPE;
 			break;
 		default:
 			Assert(false);
 			break;
 	}
 
+	return DEPARSE_NODE_CONTEXT_NONE;
+}
+
+static void deparseAlterTableMoveAllStmt(StringInfo str, AlterTableMoveAllStmt *move_all_stmt)
+{
+	appendStringInfoString(str, "ALTER ");
+	deparseAlterTableObjType(str, move_all_stmt->objtype);
+
+	appendStringInfoString(str, "ALL IN TABLESPACE ");
+	appendStringInfoString(str, move_all_stmt->orig_tablespacename);
+	appendStringInfoChar(str, ' ');
+
+	if (move_all_stmt->roles)
+	{
+		appendStringInfoString(str, "OWNED BY ");
+		deparseRoleList(str, move_all_stmt->roles);
+		appendStringInfoChar(str, ' ');
+	}
+
+	appendStringInfoString(str, "SET TABLESPACE ");
+	appendStringInfoString(str, move_all_stmt->new_tablespacename);
+	appendStringInfoChar(str, ' ');
+
+	if (move_all_stmt->nowait)
+	{
+		appendStringInfoString(str, "NOWAIT");
+	}
+}
+
+static void deparseAlterTableStmt(StringInfo str, AlterTableStmt *alter_table_stmt)
+{
+	ListCell *lc;
+
+	appendStringInfoString(str, "ALTER ");
+	DeparseNodeContext context = deparseAlterTableObjType(str, alter_table_stmt->objtype);
+
 	if (alter_table_stmt->missing_ok)
 		appendStringInfoString(str, "IF EXISTS ");
 
@@ -6458,28 +7003,7 @@ static void deparseVacuumStmt(StringInfo str, VacuumStmt *vacuum_stmt)
 	else
 		appendStringInfoString(str, "ANALYZE ");
 
-	if (list_length(vacuum_stmt->options) > 0)
-	{
-		appendStringInfoChar(str, '(');
-		foreach(lc, vacuum_stmt->options)
-		{
-			DefElem *def_elem = castNode(DefElem, lfirst(lc));
-			deparseGenericDefElemName(str, def_elem->defname);
-			if (def_elem->arg != NULL)
-			{
-				appendStringInfoChar(str, ' ');
-				if (IsA(def_elem->arg, Integer) || IsA(def_elem->arg, Float))
-					deparseNumericOnly(str, (Value *) def_elem->arg);
-				else if (IsA(def_elem->arg, String))
-					deparseOptBooleanOrString(str, strVal(def_elem->arg));
-				else
-					Assert(false);
-			}
-			if (lnext(vacuum_stmt->options, lc))
-				appendStringInfoString(str, ", ");
-		}
-		appendStringInfoString(str, ") ");
-	}
+        deparseUtilityOptionList(str, vacuum_stmt->options);
 
 	foreach(lc, vacuum_stmt->rels)
 	{
@@ -6590,31 +7114,7 @@ static void deparseExplainStmt(StringInfo str, ExplainStmt *explain_stmt)
 
 	appendStringInfoString(str, "EXPLAIN ");
 
-	if (list_length(explain_stmt->options) > 0)
-	{
-		appendStringInfoChar(str, '(');
-
-		foreach(lc, explain_stmt->options)
-		{
-			DefElem *def_elem = castNode(DefElem, lfirst(lc));
-			deparseGenericDefElemName(str, def_elem->defname);
-
-			if (def_elem->arg != NULL && IsA(def_elem->arg, String))
-			{
-				appendStringInfoChar(str, ' ');
-				deparseOptBooleanOrString(str, strVal(def_elem->arg));
-			}
-			else if (def_elem->arg != NULL && (IsA(def_elem->arg, Integer) || IsA(def_elem->arg, Float)))
-			{
-				appendStringInfoChar(str, ' ');
-				deparseNumericOnly(str, (Value *) def_elem->arg);
-			}
-
-			if (lnext(explain_stmt->options, lc))
-				appendStringInfoString(str, ", ");
-		}
-		appendStringInfoString(str, ") ");
-	}
+        deparseUtilityOptionList(str, explain_stmt->options);
 
 	deparseExplainableStmt(str, explain_stmt->query);
 }
@@ -6668,130 +7168,194 @@ static void deparseCopyStmt(StringInfo str, CopyStmt *copy_stmt)
 
 	if (list_length(copy_stmt->options) > 0)
 	{
-		appendStringInfoString(str, "WITH (");
+		// In some cases, equivalent expressions may have slightly different parse trees for `COPY`
+		// statements. For example the following two statements result in different (but equivalent) parse
+		// trees:
+		//
+		//   - COPY foo FROM STDIN CSV FREEZE
+		//   - COPY foo FROM STDIN WITH (FORMAT CSV, FREEZE)
+		//
+		// In order to make sure we deparse to the "correct" version, we always try to deparse to the older
+		// compact syntax first.
+		//
+		// The old syntax can be seen here in the Postgres 8.4 Reference:
+		//     https://www.postgresql.org/docs/8.4/sql-copy.html
+
+		bool old_fmt = true;
+
+		// Loop over the options to see if any require the new `WITH (...)` syntax.
 		foreach(lc, copy_stmt->options)
 		{
 			DefElem *def_elem = castNode(DefElem, lfirst(lc));
 
-			if (strcmp(def_elem->defname, "format") == 0)
-			{
-				appendStringInfoString(str, "FORMAT ");
-
-				char *format = strVal(def_elem->arg);
-				if (strcmp(format, "binary") == 0)
-					appendStringInfoString(str, "BINARY");
-				else if (strcmp(format, "csv") == 0)
-					appendStringInfoString(str, "CSV");
-				else
-					Assert(false);
-			}
-			else if (strcmp(def_elem->defname, "freeze") == 0 && (def_elem->arg == NULL || intVal(def_elem->arg) == 1))
-			{
-				appendStringInfoString(str, "FREEZE");
-				if (def_elem->arg != NULL && intVal(def_elem->arg) == 1)
-					appendStringInfoString(str, " 1");
-			}
-			else if (strcmp(def_elem->defname, "delimiter") == 0)
-			{
-				appendStringInfoString(str, "DELIMITER ");
-				deparseStringLiteral(str, strVal(def_elem->arg));
-			}
-			else if (strcmp(def_elem->defname, "null") == 0)
-			{
-				appendStringInfoString(str, "NULL ");
-				deparseStringLiteral(str, strVal(def_elem->arg));
-			}
-			else if (strcmp(def_elem->defname, "header") == 0 && (def_elem->arg == NULL || intVal(def_elem->arg) == 1))
-			{
-				appendStringInfoString(str, "HEADER");
-				if (def_elem->arg != NULL && intVal(def_elem->arg) == 1)
-					appendStringInfoString(str, " 1");
-			}
-			else if (strcmp(def_elem->defname, "quote") == 0)
-			{
-				appendStringInfoString(str, "QUOTE ");
-				deparseStringLiteral(str, strVal(def_elem->arg));
-			}
-			else if (strcmp(def_elem->defname, "escape") == 0)
+			if (strcmp(def_elem->defname, "freeze") == 0 && optBooleanValue(def_elem->arg))
+			{}
+			else if (strcmp(def_elem->defname, "header") == 0 && def_elem->arg && optBooleanValue(def_elem->arg))
+			{}
+			else if (strcmp(def_elem->defname, "format") == 0 && strcmp(strVal(def_elem->arg), "csv") == 0)
+			{}
+			else if (strcmp(def_elem->defname, "force_quote") == 0 && def_elem->arg && nodeTag(def_elem->arg) == T_List)
+			{}
+			else
 			{
-				appendStringInfoString(str, "ESCAPE ");
-				deparseStringLiteral(str, strVal(def_elem->arg));
+				old_fmt = false;
+				break;
 			}
-			else if (strcmp(def_elem->defname, "force_quote") == 0)
+		}
+
+		// Branch to differing output modes, depending on if we can use the old syntax.
+		if (old_fmt) {
+			foreach(lc, copy_stmt->options)
 			{
-				appendStringInfoString(str, "FORCE_QUOTE ");
-				if (IsA(def_elem->arg, A_Star))
+				DefElem *def_elem = castNode(DefElem, lfirst(lc));
+
+				if (strcmp(def_elem->defname, "freeze") == 0 && optBooleanValue(def_elem->arg))
 				{
-					appendStringInfoChar(str, '*');
+					appendStringInfoString(str, "FREEZE ");
 				}
-				else if (IsA(def_elem->arg, List))
+				else if (strcmp(def_elem->defname, "header") == 0 && def_elem->arg && optBooleanValue(def_elem->arg))
 				{
-					appendStringInfoChar(str, '(');
+					appendStringInfoString(str, "HEADER ");
+				}
+				else if (strcmp(def_elem->defname, "format") == 0 && strcmp(strVal(def_elem->arg), "csv") == 0)
+				{
+					appendStringInfoString(str, "CSV ");
+				}
+				else if (strcmp(def_elem->defname, "force_quote") == 0 && def_elem->arg && nodeTag(def_elem->arg) == T_List)
+				{
+					appendStringInfoString(str, "FORCE QUOTE ");
 					deparseColumnList(str, castNode(List, def_elem->arg));
-					appendStringInfoChar(str, ')');
 				}
 				else
 				{
+					// This isn't reachable, the conditions here are exactly the same as the first loop above.
 					Assert(false);
 				}
 			}
-			else if (strcmp(def_elem->defname, "force_not_null") == 0)
+		} else {
+			appendStringInfoString(str, "WITH (");
+			foreach(lc, copy_stmt->options)
 			{
-				appendStringInfoString(str, "FORCE_NOT_NULL (");
-				deparseColumnList(str, castNode(List, def_elem->arg));
-				appendStringInfoChar(str, ')');
-			}
-			else if (strcmp(def_elem->defname, "force_null") == 0)
-			{
-				appendStringInfoString(str, "FORCE_NULL (");
-				deparseColumnList(str, castNode(List, def_elem->arg));
-				appendStringInfoChar(str, ')');
-			}
-			else if (strcmp(def_elem->defname, "encoding") == 0)
-			{
-				appendStringInfoString(str, "ENCODING ");
-				deparseStringLiteral(str, strVal(def_elem->arg));
-			}
-			else
-			{
-				appendStringInfoString(str, quote_identifier(def_elem->defname));
-				if (def_elem->arg != NULL)
-					appendStringInfoChar(str, ' ');
-				
-				if (def_elem->arg == NULL)
+				DefElem *def_elem = castNode(DefElem, lfirst(lc));
+
+				if (strcmp(def_elem->defname, "format") == 0)
 				{
-					// Nothing
+					appendStringInfoString(str, "FORMAT ");
+
+					char *format = strVal(def_elem->arg);
+					if (strcmp(format, "binary") == 0)
+						appendStringInfoString(str, "BINARY");
+					else if (strcmp(format, "csv") == 0)
+						appendStringInfoString(str, "CSV");
+					else
+						Assert(false);
 				}
-				else if (IsA(def_elem->arg, String))
+				else if (strcmp(def_elem->defname, "freeze") == 0)
 				{
-					deparseOptBooleanOrString(str, strVal(def_elem->arg));
+					appendStringInfoString(str, "FREEZE");
+					deparseOptBoolean(str, def_elem->arg);
 				}
-				else if (IsA(def_elem->arg, Integer) || IsA(def_elem->arg, Float))
+				else if (strcmp(def_elem->defname, "delimiter") == 0)
 				{
-					deparseNumericOnly(str, (Value *) def_elem->arg);
+					appendStringInfoString(str, "DELIMITER ");
+					deparseStringLiteral(str, strVal(def_elem->arg));
 				}
-				else if (IsA(def_elem->arg, A_Star))
+				else if (strcmp(def_elem->defname, "null") == 0)
 				{
-					deparseAStar(str, castNode(A_Star, def_elem->arg));
+					appendStringInfoString(str, "NULL ");
+					deparseStringLiteral(str, strVal(def_elem->arg));
 				}
-				else if (IsA(def_elem->arg, List))
+				else if (strcmp(def_elem->defname, "header") == 0)
 				{
-					List *l = castNode(List, def_elem->arg);
-					appendStringInfoChar(str, '(');
-					foreach(lc2, l)
+					appendStringInfoString(str, "HEADER");
+					deparseOptBoolean(str, def_elem->arg);
+				}
+				else if (strcmp(def_elem->defname, "quote") == 0)
+				{
+					appendStringInfoString(str, "QUOTE ");
+					deparseStringLiteral(str, strVal(def_elem->arg));
+				}
+				else if (strcmp(def_elem->defname, "escape") == 0)
+				{
+					appendStringInfoString(str, "ESCAPE ");
+					deparseStringLiteral(str, strVal(def_elem->arg));
+				}
+				else if (strcmp(def_elem->defname, "force_quote") == 0)
+				{
+					appendStringInfoString(str, "FORCE_QUOTE ");
+					if (IsA(def_elem->arg, A_Star))
+					{
+						appendStringInfoChar(str, '*');
+					}
+					else if (IsA(def_elem->arg, List))
+					{
+						appendStringInfoChar(str, '(');
+						deparseColumnList(str, castNode(List, def_elem->arg));
+						appendStringInfoChar(str, ')');
+					}
+					else
 					{
-						deparseOptBooleanOrString(str, strVal(lfirst(lc2)));
-						if (lnext(l, lc2))
-							appendStringInfoString(str, ", ");
+						Assert(false);
 					}
+				}
+				else if (strcmp(def_elem->defname, "force_not_null") == 0)
+				{
+					appendStringInfoString(str, "FORCE_NOT_NULL (");
+					deparseColumnList(str, castNode(List, def_elem->arg));
 					appendStringInfoChar(str, ')');
 				}
-			}
+				else if (strcmp(def_elem->defname, "force_null") == 0)
+				{
+					appendStringInfoString(str, "FORCE_NULL (");
+					deparseColumnList(str, castNode(List, def_elem->arg));
+					appendStringInfoChar(str, ')');
+				}
+				else if (strcmp(def_elem->defname, "encoding") == 0)
+				{
+					appendStringInfoString(str, "ENCODING ");
+					deparseStringLiteral(str, strVal(def_elem->arg));
+				}
+				else
+				{
+					appendStringInfoString(str, quote_identifier(def_elem->defname));
+					if (def_elem->arg != NULL)
+						appendStringInfoChar(str, ' ');
+					
+					if (def_elem->arg == NULL)
+					{
+						// Nothing
+					}
+					else if (IsA(def_elem->arg, String))
+					{
+						deparseOptBooleanOrString(str, strVal(def_elem->arg));
+					}
+					else if (IsA(def_elem->arg, Integer) || IsA(def_elem->arg, Float))
+					{
+						deparseNumericOnly(str, (union ValUnion *) def_elem->arg);
+					}
+					else if (IsA(def_elem->arg, A_Star))
+					{
+						deparseAStar(str, castNode(A_Star, def_elem->arg));
+					}
+					else if (IsA(def_elem->arg, List))
+					{
+						List *l = castNode(List, def_elem->arg);
+						appendStringInfoChar(str, '(');
+						foreach(lc2, l)
+						{
+							deparseOptBooleanOrString(str, strVal(lfirst(lc2)));
+							if (lnext(l, lc2))
+								appendStringInfoString(str, ", ");
+						}
+						appendStringInfoChar(str, ')');
+					}
+				}
 
-			if (lnext(copy_stmt->options, lc))
-				appendStringInfoString(str, ", ");
+				if (lnext(copy_stmt->options, lc))
+					appendStringInfoString(str, ", ");
+			}
+			appendStringInfoString(str, ") ");
 		}
-		appendStringInfoString(str, ") ");
 	}
 
 	deparseWhereClause(str, copy_stmt->whereClause);
@@ -7248,7 +7812,6 @@ static void deparseAccessPriv(StringInfo str, AccessPriv *access_priv)
 static void deparseGrantStmt(StringInfo str, GrantStmt *grant_stmt)
 {
 	ListCell *lc;
-
 	if (grant_stmt->is_grant)
 		appendStringInfoString(str, "GRANT ");
 	else
@@ -7295,6 +7858,12 @@ static void deparseGrantStmt(StringInfo str, GrantStmt *grant_stmt)
 
 	deparseOptDropBehavior(str, grant_stmt->behavior);
 
+	if (grant_stmt->grantor)
+	{
+		appendStringInfoString(str, "GRANTED BY ");
+		deparseRoleSpec(str, castNode(RoleSpec, grant_stmt->grantor));
+	}
+
 	removeTrailingSpace(str);
 }
 
@@ -7307,6 +7876,9 @@ static void deparseGrantRoleStmt(StringInfo str, GrantRoleStmt *grant_role_stmt)
 	else
 		appendStringInfoString(str, "REVOKE ");
 
+	if (!grant_role_stmt->is_grant && grant_role_stmt->admin_opt)
+		appendStringInfoString(str, "ADMIN OPTION FOR ");
+
 	foreach(lc, grant_role_stmt->granted_roles)
 	{
 		deparseAccessPriv(str, castNode(AccessPriv, lfirst(lc)));
@@ -7323,9 +7895,15 @@ static void deparseGrantRoleStmt(StringInfo str, GrantRoleStmt *grant_role_stmt)
 	deparseRoleList(str, grant_role_stmt->grantee_roles);
 	appendStringInfoChar(str, ' ');
 
-	if (grant_role_stmt->admin_opt)
+	if (grant_role_stmt->is_grant && grant_role_stmt->admin_opt)
 		appendStringInfoString(str, "WITH ADMIN OPTION ");
 
+	if (grant_role_stmt->grantor)
+	{
+		appendStringInfoString(str, "GRANTED BY ");
+		deparseRoleSpec(str, castNode(RoleSpec, grant_role_stmt->grantor));
+	}
+
 	removeTrailingSpace(str);
 }
 
@@ -7396,6 +7974,11 @@ static void deparseIndexStmt(StringInfo str, IndexStmt *index_stmt)
 		appendStringInfoString(str, ") ");
 	}
 
+	if (index_stmt->nulls_not_distinct)
+	{
+		appendStringInfoString(str, "NULLS NOT DISTINCT ");
+	}
+
 	deparseOptWith(str, index_stmt->options);
 
 	if (index_stmt->tableSpace != NULL)
@@ -7499,59 +8082,59 @@ static void deparseAlterRoleElem(StringInfo str, DefElem *def_elem)
 		appendStringInfoString(str, "VALID UNTIL ");
 		deparseStringLiteral(str, strVal(def_elem->arg));
 	}
-	else if (strcmp(def_elem->defname, "superuser") == 0 && intVal(def_elem->arg) == 1)
+	else if (strcmp(def_elem->defname, "superuser") == 0 && boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "SUPERUSER");
 	}
-	else if (strcmp(def_elem->defname, "superuser") == 0 && intVal(def_elem->arg) == 0)
+	else if (strcmp(def_elem->defname, "superuser") == 0 && !boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "NOSUPERUSER");
 	}
-	else if (strcmp(def_elem->defname, "createrole") == 0 && intVal(def_elem->arg) == 1)
+	else if (strcmp(def_elem->defname, "createrole") == 0 && boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "CREATEROLE");
 	}
-	else if (strcmp(def_elem->defname, "createrole") == 0 && intVal(def_elem->arg) == 0)
+	else if (strcmp(def_elem->defname, "createrole") == 0 && !boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "NOCREATEROLE");
 	}
-	else if (strcmp(def_elem->defname, "isreplication") == 0 && intVal(def_elem->arg) == 1)
+	else if (strcmp(def_elem->defname, "isreplication") == 0 && boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "REPLICATION");
 	}
-	else if (strcmp(def_elem->defname, "isreplication") == 0 && intVal(def_elem->arg) == 0)
+	else if (strcmp(def_elem->defname, "isreplication") == 0 && !boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "NOREPLICATION");
 	}
-	else if (strcmp(def_elem->defname, "createdb") == 0 && intVal(def_elem->arg) == 1)
+	else if (strcmp(def_elem->defname, "createdb") == 0 && boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "CREATEDB");
 	}
-	else if (strcmp(def_elem->defname, "createdb") == 0 && intVal(def_elem->arg) == 0)
+	else if (strcmp(def_elem->defname, "createdb") == 0 && !boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "NOCREATEDB");
 	}
-	else if (strcmp(def_elem->defname, "canlogin") == 0 && intVal(def_elem->arg) == 1)
+	else if (strcmp(def_elem->defname, "canlogin") == 0 && boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "LOGIN");
 	}
-	else if (strcmp(def_elem->defname, "canlogin") == 0 && intVal(def_elem->arg) == 0)
+	else if (strcmp(def_elem->defname, "canlogin") == 0 && !boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "NOLOGIN");
 	}
-	else if (strcmp(def_elem->defname, "bypassrls") == 0 && intVal(def_elem->arg) == 1)
+	else if (strcmp(def_elem->defname, "bypassrls") == 0 && boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "BYPASSRLS");
 	}
-	else if (strcmp(def_elem->defname, "bypassrls") == 0 && intVal(def_elem->arg) == 0)
+	else if (strcmp(def_elem->defname, "bypassrls") == 0 && !boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "NOBYPASSRLS");
 	}
-	else if (strcmp(def_elem->defname, "inherit") == 0 && intVal(def_elem->arg) == 1)
+	else if (strcmp(def_elem->defname, "inherit") == 0 && boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "INHERIT");
 	}
-	else if (strcmp(def_elem->defname, "inherit") == 0 && intVal(def_elem->arg) == 0)
+	else if (strcmp(def_elem->defname, "inherit") == 0 && !boolVal(def_elem->arg))
 	{
 		appendStringInfoString(str, "NOINHERIT");
 	}
@@ -7824,8 +8407,7 @@ static void deparseReindexStmt(StringInfo str, ReindexStmt *reindex_stmt)
 {
 	appendStringInfoString(str, "REINDEX ");
 
-	if (reindex_stmt->options & REINDEXOPT_VERBOSE)
-		appendStringInfoString(str, "(VERBOSE) ");
+        deparseUtilityOptionList(str, reindex_stmt->params);
 
 	switch (reindex_stmt->kind)
 	{
@@ -7846,9 +8428,6 @@ static void deparseReindexStmt(StringInfo str, ReindexStmt *reindex_stmt)
 			break;
 	}
 
-	if (reindex_stmt->concurrent)
-		appendStringInfoString(str, "CONCURRENTLY ");
-
 	if (reindex_stmt->relation != NULL)
 	{
 		deparseRangeVar(str, reindex_stmt->relation, DEPARSE_NODE_CONTEXT_NONE);
@@ -7892,6 +8471,9 @@ static void deparseRuleStmt(StringInfo str, RuleStmt* rule_stmt)
 		case CMD_DELETE:
 			appendStringInfoString(str, "DELETE ");
 			break;
+		case CMD_MERGE:
+			appendStringInfoString(str, "MERGE ");
+			break;
 	}
 
 	appendStringInfoString(str, "TO ");
@@ -8214,7 +8796,12 @@ static void deparseCreateTableSpaceStmt(StringInfo str, CreateTableSpaceStmt *cr
 	}
 
 	appendStringInfoString(str, "LOCATION ");
-	deparseStringLiteral(str, create_table_space_stmt->location);
+
+	if (create_table_space_stmt->location != NULL)
+		deparseStringLiteral(str, create_table_space_stmt->location);
+	else
+		appendStringInfoString(str, "''");
+
 	appendStringInfoChar(str, ' ');
 
 	deparseOptWith(str, create_table_space_stmt->options);
@@ -8277,6 +8864,50 @@ static void deparseCreateAmStmt(StringInfo str, CreateAmStmt *create_am_stmt)
 	deparseHandlerName(str, create_am_stmt->handler_name);
 }
 
+static void deparsePublicationObjectList(StringInfo str, List *pubobjects) {
+	const ListCell *lc;
+	foreach(lc, pubobjects) {
+		PublicationObjSpec *obj = lfirst(lc);
+
+		switch (obj->pubobjtype) {
+			case PUBLICATIONOBJ_TABLE:
+				appendStringInfoString(str, "TABLE ");
+				deparseRangeVar(str, obj->pubtable->relation, DEPARSE_NODE_CONTEXT_NONE);
+				
+				if (obj->pubtable->columns)
+				{
+					appendStringInfoChar(str, '(');
+					deparseColumnList(str, obj->pubtable->columns);
+					appendStringInfoChar(str, ')');
+				}
+
+				if (obj->pubtable->whereClause)
+				{
+					appendStringInfoString(str, " WHERE (");
+					deparseExpr(str, obj->pubtable->whereClause);
+					appendStringInfoString(str, ")");
+				}
+
+				break;
+			case PUBLICATIONOBJ_TABLES_IN_SCHEMA:
+				appendStringInfoString(str, "TABLES IN SCHEMA ");
+				appendStringInfoString(str, quote_identifier(obj->name));
+				break;
+			case PUBLICATIONOBJ_TABLES_IN_CUR_SCHEMA:
+				appendStringInfoString(str, "TABLES IN SCHEMA CURRENT_SCHEMA");
+				break;
+			case PUBLICATIONOBJ_CONTINUATION:
+				// This should be unreachable, the parser merges these before we can even get here.
+				Assert(false);
+				break;
+		}
+		
+		if (lnext(pubobjects, lc)) {
+			appendStringInfoString(str, ", ");
+		}
+	}
+}
+
 static void deparseCreatePublicationStmt(StringInfo str, CreatePublicationStmt *create_publication_stmt)
 {
 	ListCell *lc = NULL;
@@ -8285,10 +8916,10 @@ static void deparseCreatePublicationStmt(StringInfo str, CreatePublicationStmt *
 	appendStringInfoString(str, quote_identifier(create_publication_stmt->pubname));
 	appendStringInfoChar(str, ' ');
 
-	if (list_length(create_publication_stmt->tables) > 0)
+	if (list_length(create_publication_stmt->pubobjects) > 0)
 	{
-		appendStringInfoString(str, "FOR TABLE ");
-		deparseRelationExprList(str, create_publication_stmt->tables);
+		appendStringInfoString(str, "FOR ");
+		deparsePublicationObjectList(str, create_publication_stmt->pubobjects);
 		appendStringInfoChar(str, ' ');
 	}
 	else if (create_publication_stmt->for_all_tables)
@@ -8306,25 +8937,22 @@ static void deparseAlterPublicationStmt(StringInfo str, AlterPublicationStmt *al
 	deparseColId(str, alter_publication_stmt->pubname);
 	appendStringInfoChar(str, ' ');
 
-	if (list_length(alter_publication_stmt->tables) > 0)
+	if (list_length(alter_publication_stmt->pubobjects) > 0)
 	{
-		switch (alter_publication_stmt->tableAction)
+		switch (alter_publication_stmt->action)
 		{
-			case DEFELEM_SET:
-				appendStringInfoString(str, "SET TABLE ");
-				break;
-			case DEFELEM_ADD:
-				appendStringInfoString(str, "ADD TABLE ");
+			case AP_SetObjects:
+				appendStringInfoString(str, "SET ");
 				break;
-			case DEFELEM_DROP:
-				appendStringInfoString(str, "DROP TABLE ");
+			case AP_AddObjects:
+				appendStringInfoString(str, "ADD ");
 				break;
-			case DEFELEM_UNSPEC:
-				Assert(false);
+			case AP_DropObjects:
+				appendStringInfoString(str, "DROP ");
 				break;
 		}
 
-		deparseRelationExprList(str, alter_publication_stmt->tables);
+		deparsePublicationObjectList(str, alter_publication_stmt->pubobjects);
 	}
 	else if (list_length(alter_publication_stmt->options) > 0)
 	{
@@ -8581,7 +9209,7 @@ static void deparseCommentStmt(StringInfo str, CommentStmt *comment_stmt)
 			appendStringInfoString(str, quote_identifier(strVal(linitial(l))));
 			break;
 		case OBJECT_LARGEOBJECT:
-			deparseValue(str, (Value *) comment_stmt->object, DEPARSE_NODE_CONTEXT_NONE);
+			deparseValue(str, (union ValUnion *) comment_stmt->object, DEPARSE_NODE_CONTEXT_NONE);
 			break;
 		case OBJECT_CAST:
 			l = castNode(List, comment_stmt->object);
@@ -8605,6 +9233,19 @@ static void deparseCommentStmt(StringInfo str, CommentStmt *comment_stmt)
 		appendStringInfoString(str, "NULL");
 }
 
+static void deparseStatsElem(StringInfo str, StatsElem *stats_elem)
+{
+	// only one of stats_elem->name or stats_elem->expr can be non-null
+	if (stats_elem->name)
+		appendStringInfoString(str, stats_elem->name);
+	else if (stats_elem->expr)
+	{
+		appendStringInfoChar(str, '(');
+		deparseExpr(str, stats_elem->expr);
+		appendStringInfoChar(str, ')');
+	}
+}
+
 static void deparseCreateStatsStmt(StringInfo str, CreateStatsStmt *create_stats_stmt)
 {
 	ListCell *lc;
@@ -8625,7 +9266,12 @@ static void deparseCreateStatsStmt(StringInfo str, CreateStatsStmt *create_stats
 	}
 
 	appendStringInfoString(str, "ON ");
-	deparseExprList(str, create_stats_stmt->exprs);
+	foreach (lc, create_stats_stmt->exprs)
+	{
+		deparseStatsElem(str, lfirst(lc));
+		if (lnext(create_stats_stmt->exprs, lc))
+			appendStringInfoString(str, ", ");
+	}
 
 	appendStringInfoString(str, " FROM ");
 	deparseFromList(str, create_stats_stmt->relations);
@@ -8737,7 +9383,7 @@ static void deparseVariableShowStmt(StringInfo str, VariableShowStmt *variable_s
 	else if (strcmp(variable_show_stmt->name, "all") == 0)
 		appendStringInfoString(str, "SESSION ALL");
 	else
-		appendStringInfoString(str, variable_show_stmt->name);
+		appendStringInfoString(str, quote_identifier(variable_show_stmt->name));
 }
 
 static void deparseRangeTableSample(StringInfo str, RangeTableSample *range_table_sample)
@@ -8802,6 +9448,10 @@ static void deparseAlterSubscriptionStmt(StringInfo str, AlterSubscriptionStmt *
 			appendStringInfoString(str, "SET ");
 			deparseDefinition(str, alter_subscription_stmt->options);
 			break;
+		case ALTER_SUBSCRIPTION_SKIP:
+			appendStringInfoString(str, "SKIP ");
+			deparseDefinition(str, alter_subscription_stmt->options);
+			break;
 		case ALTER_SUBSCRIPTION_CONNECTION:
 			appendStringInfoString(str, "CONNECTION ");
 			deparseStringLiteral(str, alter_subscription_stmt->conninfo);
@@ -8811,7 +9461,29 @@ static void deparseAlterSubscriptionStmt(StringInfo str, AlterSubscriptionStmt *
 			appendStringInfoString(str, "REFRESH PUBLICATION ");
 			deparseOptDefinition(str, alter_subscription_stmt->options);
 			break;
-		case ALTER_SUBSCRIPTION_PUBLICATION:
+		case ALTER_SUBSCRIPTION_ADD_PUBLICATION:
+			appendStringInfoString(str, "ADD PUBLICATION ");
+			foreach(lc, alter_subscription_stmt->publication)
+			{
+				deparseColLabel(str, strVal(lfirst(lc)));
+				if (lnext(alter_subscription_stmt->publication, lc))
+					appendStringInfoString(str, ", ");
+			}
+			appendStringInfoChar(str, ' ');
+			deparseOptDefinition(str, alter_subscription_stmt->options);
+			break;
+		case ALTER_SUBSCRIPTION_DROP_PUBLICATION:
+			appendStringInfoString(str, "DROP PUBLICATION ");
+			foreach(lc, alter_subscription_stmt->publication)
+			{
+				deparseColLabel(str, strVal(lfirst(lc)));
+				if (lnext(alter_subscription_stmt->publication, lc))
+					appendStringInfoString(str, ", ");
+			}
+			appendStringInfoChar(str, ' ');
+			deparseOptDefinition(str, alter_subscription_stmt->options);
+			break;
+		case ALTER_SUBSCRIPTION_SET_PUBLICATION:
 			appendStringInfoString(str, "SET PUBLICATION ");
 			foreach(lc, alter_subscription_stmt->publication)
 			{
@@ -8826,17 +9498,13 @@ static void deparseAlterSubscriptionStmt(StringInfo str, AlterSubscriptionStmt *
 			Assert(list_length(alter_subscription_stmt->options) == 1);
 			DefElem *defelem = castNode(DefElem, linitial(alter_subscription_stmt->options));
 			Assert(strcmp(defelem->defname, "enabled") == 0);
-			if (intVal(defelem->arg) == 1)
+			if (optBooleanValue(defelem->arg))
 			{
 				appendStringInfoString(str, " ENABLE ");
 			}
-			else if (intVal(defelem->arg) == 0)
-			{
-				appendStringInfoString(str, " DISABLE ");
-			}
 			else
 			{
-				Assert(false);
+				appendStringInfoString(str, " DISABLE ");
 			}
 			break;
 	}
@@ -8898,7 +9566,7 @@ static void deparseAlterOwnerStmt(StringInfo str, AlterOwnerStmt *alter_owner_st
 			break;
 		case OBJECT_LARGEOBJECT:
 			appendStringInfoString(str, "LARGE OBJECT ");
-			deparseNumericOnly(str, (Value *) alter_owner_stmt->object);
+			deparseNumericOnly(str, (union ValUnion *) alter_owner_stmt->object);
 			break;
 		case OBJECT_OPERATOR:
 			appendStringInfoString(str, "OPERATOR ");
@@ -9061,6 +9729,8 @@ static void deparseCreateTrigStmt(StringInfo str, CreateTrigStmt *create_trig_st
 	bool skip_events_or = true;
 
 	appendStringInfoString(str, "CREATE ");
+	if (create_trig_stmt->replace)
+		appendStringInfoString(str, "OR REPLACE ");
 	if (create_trig_stmt->isconstraint)
 		appendStringInfoString(str, "CONSTRAINT ");
 	appendStringInfoString(str, "TRIGGER ");
@@ -9224,8 +9894,6 @@ static void deparseXmlExpr(StringInfo str, XmlExpr* xml_expr)
 					Assert(false);
 			}
 			deparseExpr(str, linitial(xml_expr->args));
-			if (strcmp(strVal(&castNode(A_Const, castNode(TypeCast, lsecond(xml_expr->args))->arg)->val), "t") == 0)
-				appendStringInfoString(str, " PRESERVE WHITESPACE");
 			appendStringInfoChar(str, ')');
 			break;
 		case IS_XMLPI: /* XMLPI(name [, args]) */
@@ -9242,7 +9910,7 @@ static void deparseXmlExpr(StringInfo str, XmlExpr* xml_expr)
 			appendStringInfoString(str, "xmlroot(");
 			deparseExpr(str, linitial(xml_expr->args));
 			appendStringInfoString(str, ", version ");
-			if (nodeTag(&castNode(A_Const, lsecond(xml_expr->args))->val) == T_Null)
+			if (castNode(A_Const, lsecond(xml_expr->args))->isnull)
 				appendStringInfoString(str, "NO VALUE");
 			else
 				deparseExpr(str, lsecond(xml_expr->args));
@@ -9372,8 +10040,8 @@ static void deparseGroupingFunc(StringInfo str, GroupingFunc *grouping_func)
 static void deparseClusterStmt(StringInfo str, ClusterStmt *cluster_stmt)
 {
 	appendStringInfoString(str, "CLUSTER ");
-	if (cluster_stmt->options & CLUOPT_VERBOSE)
-		appendStringInfoString(str, "VERBOSE ");
+
+        deparseUtilityOptionList(str, cluster_stmt->params);
 
 	if (cluster_stmt->relation != NULL)
 	{
@@ -9391,42 +10059,47 @@ static void deparseClusterStmt(StringInfo str, ClusterStmt *cluster_stmt)
 	removeTrailingSpace(str);
 }
 
-static void deparseValue(StringInfo str, Value *value, DeparseNodeContext context)
+static void deparseValue(StringInfo str, union ValUnion *value, DeparseNodeContext context)
 {
+	if (!value) {
+		appendStringInfoString(str, "NULL");
+		return;
+	}
+
 	switch (nodeTag(value))
 	{
 		case T_Integer:
 		case T_Float:
 			deparseNumericOnly(str, value);
 			break;
+		case T_Boolean:
+			appendStringInfoString(str, value->boolval.boolval ? "true" : "false");
+			break;
 		case T_String:
 			if (context == DEPARSE_NODE_CONTEXT_IDENTIFIER) {
-				appendStringInfoString(str, quote_identifier(value->val.str));
+				appendStringInfoString(str, quote_identifier(value->sval.sval));
 			} else if (context == DEPARSE_NODE_CONTEXT_CONSTANT) {
-				deparseStringLiteral(str, value->val.str);
+				deparseStringLiteral(str, value->sval.sval);
 			} else {
-				appendStringInfoString(str, value->val.str);
+				appendStringInfoString(str, value->sval.sval);
 			}
 			break;
 		case T_BitString:
-			if (strlen(value->val.str) >= 1 && value->val.str[0] == 'x')
+			if (strlen(value->sval.sval) >= 1 && value->sval.sval[0] == 'x')
 			{
 				appendStringInfoChar(str, 'x');
-				deparseStringLiteral(str, value->val.str + 1);
+				deparseStringLiteral(str, value->sval.sval + 1);
 			}
-			else if (strlen(value->val.str) >= 1 && value->val.str[0] == 'b')
+			else if (strlen(value->sval.sval) >= 1 && value->sval.sval[0] == 'b')
 			{
 				appendStringInfoChar(str, 'b');
-				deparseStringLiteral(str, value->val.str + 1);
+				deparseStringLiteral(str, value->sval.sval + 1);
 			}
 			else
 			{
 				Assert(false);
 			}
 			break;
-		case T_Null:
-			appendStringInfoString(str, "NULL");
-			break;
 		default:
 			elog(ERROR, "deparse: unrecognized value node type: %d",
 				 (int) nodeTag(value));
@@ -9451,6 +10124,9 @@ static void deparsePreparableStmt(StringInfo str, Node *node)
 		case T_DeleteStmt:
 			deparseDeleteStmt(str, castNode(DeleteStmt, node));
 			break;
+		case T_MergeStmt:
+			deparseMergeStmt(str, castNode(MergeStmt, node));
+			break;
 		default:
 			Assert(false);
 	}
@@ -9510,6 +10186,9 @@ static void deparseExplainableStmt(StringInfo str, Node *node)
 		case T_ExecuteStmt:
 			deparseExecuteStmt(str, castNode(ExecuteStmt, node));
 			break;
+		case T_MergeStmt:
+			deparseMergeStmt(str, castNode(MergeStmt, node));
+			break;
 		default:
 			Assert(false);
 	}
@@ -9632,6 +10311,9 @@ static void deparseStmt(StringInfo str, Node *node)
 		case T_AlterSystemStmt:
 			deparseAlterSystemStmt(str, castNode(AlterSystemStmt, node));
 			break;
+		case T_AlterTableMoveAllStmt:
+			deparseAlterTableMoveAllStmt(str, castNode(AlterTableMoveAllStmt, node));
+			break;
 		case T_AlterTableStmt:
 			deparseAlterTableStmt(str, castNode(AlterTableStmt, node));
 			break;
@@ -9842,6 +10524,9 @@ static void deparseStmt(StringInfo str, Node *node)
 		case T_LockStmt:
 			deparseLockStmt(str, castNode(LockStmt, node));
 			break;
+		case T_MergeStmt:
+			deparseMergeStmt(str, castNode(MergeStmt, node));
+			break;
 		case T_NotifyStmt:
 			deparseNotifyStmt(str, castNode(NotifyStmt, node));
 			break;