diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2003-08-07 19:20:24 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2003-08-07 19:20:24 +0000 |
| commit | ecbed6e1b9a373d016c4afb3b674c88e86c1fc28 (patch) | |
| tree | 594b8944544d0cb1beb58a20ea8d5badd5b7f59c /src/backend/parser/parse_clause.c | |
| parent | d862045dfc9ae86358a38749863a5f94f7c2e834 (diff) | |
| download | postgresql-ecbed6e1b9a373d016c4afb3b674c88e86c1fc28.tar.gz postgresql-ecbed6e1b9a373d016c4afb3b674c88e86c1fc28.zip | |
create_unique_plan() should not discard existing output columns of the
subplan it starts with, as they may be needed at upper join levels.
See comments added to code for the non-obvious reason why. Per bug report
from Robert Creager.
Diffstat (limited to 'src/backend/parser/parse_clause.c')
| -rw-r--r-- | src/backend/parser/parse_clause.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/src/backend/parser/parse_clause.c b/src/backend/parser/parse_clause.c index f56e06136e8..ebc3ed23eec 100644 --- a/src/backend/parser/parse_clause.c +++ b/src/backend/parser/parse_clause.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/parser/parse_clause.c,v 1.120 2003/08/04 02:40:01 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/parser/parse_clause.c,v 1.121 2003/08/07 19:20:22 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -59,9 +59,6 @@ static Node *buildMergedJoinVar(ParseState *pstate, JoinType jointype, Var *l_colvar, Var *r_colvar); static TargetEntry *findTargetlistEntry(ParseState *pstate, Node *node, List *tlist, int clause); -static List *addTargetToSortList(ParseState *pstate, TargetEntry *tle, - List *sortlist, List *targetlist, - List *opname, bool resolveUnknown); /* @@ -1478,7 +1475,7 @@ addAllTargetsToSortList(ParseState *pstate, List *sortlist, * * Returns the updated ORDER BY list. */ -static List * +List * addTargetToSortList(ParseState *pstate, TargetEntry *tle, List *sortlist, List *targetlist, List *opname, bool resolveUnknown) |