Revert unnecessary change in MV call to checkRuleResultList().

kgrittn · kgrittn · commit fb60e7296c2c · 2013-03-14T13:59:52.000-05:00
Due to a misreading of the function's comment block, there was an
unneeded change to a call in rewriteDefine.c.  There is, in fact
no reason to pass false for a MV; it should be true just like a
view.

Fixes issue pointed out by Tom Lane
diff --git a/src/backend/rewrite/rewriteDefine.c b/src/backend/rewrite/rewriteDefine.c
@@ -356,8 +356,7 @@ DefineQueryRewrite(char *rulename,
 		 */
 		checkRuleResultList(query->targetList,
 							RelationGetDescr(event_relation),
-							event_relation->rd_rel->relkind !=
-								RELKIND_MATVIEW);
+							true);
 
 		/*
 		 * ... there must not be another ON SELECT rule already ...
