canonicalize user's partitioning expression in add_to_pathman_config()

funbringer · funbringer · commit 5d43c5721d3e · 2017-05-11T18:48:27.000+03:00
diff --git a/src/pl_funcs.c b/src/pl_funcs.c
@@ -32,6 +32,7 @@
 #include "nodes/nodeFuncs.h"
 #include "utils/builtins.h"
 #include "utils/inval.h"
+#include "utils/ruleutils.h"
 #include "utils/snapmgr.h"
 #include "utils/lsyscache.h"
 #include "utils/syscache.h"
@@ -781,6 +782,11 @@ add_to_pathman_config(PG_FUNCTION_ARGS)
 	/* Parse and check expression */
 	expr_datum = cook_partitioning_expression(relid, expression, &expr_type);
 
+	/* Canonicalize user's expression (trim whitespaces etc) */
+	expression = deparse_expression(stringToNode(TextDatumGetCString(expr_datum)),
+									deparse_context_for(get_rel_name(relid), relid),
+									false, false);
+
 	/* Check hash function for HASH partitioning */
 	if (parttype == PT_HASH)
 	{
