From fb60e7296c2cf15195802b4596496b179bdc905a Mon Sep 17 00:00:00 2001
From: Kevin Grittner <kgrittn@postgresql.org>
Date: Thu, 14 Mar 2013 13:59:52 -0500
Subject: [PATCH] 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
---
 src/backend/rewrite/rewriteDefine.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

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 ...