postgres.git / summary / log / commit / refs

commit    fb60e7296c2cf15195802b4596496b179bdc905a
Author:   Kevin Grittner <kgrittn@postgresql.org>
Date:     Thu Mar 14 18:59:52 2013 +0000
First released in: 9.3.0

    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 0e265db15c5..ca92fb56417 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 ...


[parent: 8d7ff13ed562]