>> Hm. I'd sort of expect the "z" to become both the table and column

>> alias in this case.  What do you think?
>
> I guess that would make sense. I'll make a separate patch just for
that
> change if that's OK.
>

Simple change -- patch attached.

test=# select * from myfoo1() as z;
  z
----
   1
   2
   3
(3 rows)

Joe Conway
This commit is contained in:
Bruce Momjian 2002-08-06 05:34:10 +00:00
parent dd6513a5b6
commit 4f63e11646

View File

@ -8,7 +8,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/parser/parse_relation.c,v 1.74 2002/08/06 05:33:29 momjian Exp $ * $Header: /cvsroot/pgsql/src/backend/parser/parse_relation.c,v 1.75 2002/08/06 05:34:10 momjian Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
@ -807,7 +807,7 @@ addRangeTableEntryForFunction(ParseState *pstate,
elog(ERROR, "Too many column aliases specified for function %s", elog(ERROR, "Too many column aliases specified for function %s",
funcname); funcname);
if (numaliases == 0) if (numaliases == 0)
eref->colnames = makeList1(makeString(funcname)); eref->colnames = makeList1(makeString(eref->aliasname));
} }
else if (functyptype == 'p' && funcrettype == RECORDOID) else if (functyptype == 'p' && funcrettype == RECORDOID)
{ {