diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2005-11-23 17:21:04 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2005-11-23 17:21:04 +0000 |
| commit | 19ff959bff054574be094e9633b69fc65c6c6a3c (patch) | |
| tree | 4cfa22ee402e237c0286c5f83abb2ab528ae1b8a /src/backend/rewrite/rewriteHandler.c | |
| parent | baa6b22fcb831490c11506b6ec140d75fe8fff81 (diff) | |
| download | postgresql-19ff959bff054574be094e9633b69fc65c6c6a3c.tar.gz postgresql-19ff959bff054574be094e9633b69fc65c6c6a3c.zip | |
Fix problems with rewriter failing to set Query.hasSubLinks when inserting
a SubLink expression into a rule query. Pre-8.1 we essentially did this
unconditionally; 8.1 tries to do it only when needed, but was missing a
couple of cases. Per report from Kyle Bateman. Add some regression test
cases covering this area.
Diffstat (limited to 'src/backend/rewrite/rewriteHandler.c')
| -rw-r--r-- | src/backend/rewrite/rewriteHandler.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/backend/rewrite/rewriteHandler.c b/src/backend/rewrite/rewriteHandler.c index afde40dc54e..45b5b9207fa 100644 --- a/src/backend/rewrite/rewriteHandler.c +++ b/src/backend/rewrite/rewriteHandler.c @@ -7,7 +7,7 @@ * Portions Copyright (c) 1994, Regents of the University of California * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/rewrite/rewriteHandler.c,v 1.159 2005/11/22 18:17:19 momjian Exp $ + * $PostgreSQL: pgsql/src/backend/rewrite/rewriteHandler.c,v 1.160 2005/11/23 17:21:03 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -374,6 +374,14 @@ rewriteRuleAction(Query *parsetree, sub_action->jointree->fromlist = list_concat(newjointree, sub_action->jointree->fromlist); + + /* + * There could have been some SubLinks in newjointree, in which + * case we'd better mark the sub_action correctly. + */ + if (parsetree->hasSubLinks && !sub_action->hasSubLinks) + sub_action->hasSubLinks = + checkExprHasSubLink((Node *) newjointree); } } |