>> 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

diff --git a/src/backend/parser/parse_relation.c b/src/backend/parser/parse_relation.c
index 6732dc5..8a15c0f 100644 (file)
--- a/src/backend/parser/parse_relation.c
+++ b/src/backend/parser/parse_relation.c
@@ -8,7 +8,7 @@
  *
  *
  * 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",
                                 funcname);
                if (numaliases == 0)
-                       eref->colnames = makeList1(makeString(funcname));
+                       eref->colnames = makeList1(makeString(eref->aliasname));
        }
        else if (functyptype == 'p' && funcrettype == RECORDOID)
        {