diff options
| author | Kevin Grittner <kgrittn@postgresql.org> | 2013-03-14 13:59:52 -0500 |
|---|---|---|
| committer | Kevin Grittner <kgrittn@postgresql.org> | 2013-03-14 13:59:52 -0500 |
| commit | fb60e7296c2cf15195802b4596496b179bdc905a (patch) | |
| tree | ff3ff40b31073be456ed03d61914dc0c87f975dc | |
| parent | 8d7ff13ed5626ecdac2bb0b356445ff4192810aa (diff) | |
| download | postgresql-fb60e7296c2cf15195802b4596496b179bdc905a.tar.gz postgresql-fb60e7296c2cf15195802b4596496b179bdc905a.zip | |
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
| -rw-r--r-- | src/backend/rewrite/rewriteDefine.c | 3 |
1 files changed, 1 insertions, 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 ... |