From: Kevin Grittner <kgrittn@postgresql.org>
Date: Thu, 14 Mar 2013 18:59:52 +0000 (-0500)
Subject: Revert unnecessary change in MV call to checkRuleResultList().
X-Git-Tag: REL9_3_BETA1~228
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=fb60e7296c2cf15195802b4596496b179bdc905a;p=postgresql

Revert unnecessary change in MV call to checkRuleResultList().

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
index 0e265db15c..ca92fb5641 100644
--- 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 ...