database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-04-24 10:47:04 +03:00
Code

RewritePreprocessQuery tried to match resjunk targets against

Browse Source 
result relation ... wrong ...
This commit is contained in:
Tom Lane 1999-07-11 17:54:30 +00:00
parent f8d856af2c
commit eeaeb1bda8
1 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/backend/rewrite/rewriteHandler.c
View File

@ -6,7 +6,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/rewrite/rewriteHandler.c,v 1.47 1999/06/21 01:26:56 tgl Exp $ * $Header: /cvsroot/pgsql/src/backend/rewrite/rewriteHandler.c,v 1.48 1999/07/11 17:54:30 tgl Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
@ -2610,6 +2610,8 @@ RewritePreprocessQuery(Query *parsetree)
* target relation. FixNew() depends on it when replacing * target relation. FixNew() depends on it when replacing
* *new* references in a rule action by the expressions * *new* references in a rule action by the expressions
* from the rewritten query. * from the rewritten query.
* resjunk targets are somewhat arbitrarily given a resno of 0;
* this is to prevent FixNew() from matching them to var nodes.
* ---------- * ----------
*/ */
if (parsetree->resultRelation > 0) if (parsetree->resultRelation > 0)
@ -2617,8 +2619,6 @@ RewritePreprocessQuery(Query *parsetree)
RangeTblEntry *rte; RangeTblEntry *rte;
Relation rd; Relation rd;
List *tl; List *tl;
TargetEntry *tle;
int resdomno;
rte = (RangeTblEntry *) nth(parsetree->resultRelation - 1, rte = (RangeTblEntry *) nth(parsetree->resultRelation - 1,
parsetree->rtable); parsetree->rtable);
@ -2626,9 +2626,12 @@ RewritePreprocessQuery(Query *parsetree)
foreach(tl, parsetree->targetList) foreach(tl, parsetree->targetList)
{ {
tle = (TargetEntry *) lfirst(tl); TargetEntry *tle = (TargetEntry *) lfirst(tl);
resdomno = attnameAttNum(rd, tle->resdom->resname);
tle->resdom->resno = resdomno; if (! tle->resdom->resjunk)
tle->resdom->resno = attnameAttNum(rd, tle->resdom->resname);
else
tle->resdom->resno = 0;
} }
heap_close(rd); heap_close(rd);