I have a simple patch about the treatment of functions.

But it may be self-satisfied.
Please check my patch at the end of this posting.

Case 1. executor evaluates functions twice

Hiroshi Inoue
Inoue@tpf.co.jp

diff --git a/src/backend/parser/parse_func.c b/src/backend/parser/parse_func.c
index e1a625f9ef2611d50489d8139fdcf6d92ff6f0e2..a95c1b46b5c897135addd79cf2d88dc0daeaffc2 100644 (file)
--- a/src/backend/parser/parse_func.c
+++ b/src/backend/parser/parse_func.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/parser/parse_func.c,v 1.32 1998/12/08 06:18:56 thomas Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/parser/parse_func.c,v 1.33 1998/12/13 04:16:51 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1490,15 +1490,13 @@ ParseComplexProjection(ParseState *pstate,
                                                newexpr->typeOid = funcnode->functype;
                                                newexpr->opType = FUNC_EXPR;
                                                newexpr->oper = (Node *) funcnode;
-                                               newexpr->args = lcons(first_arg, NIL);
+                                               newexpr->args = expr->args;
 
                                                return (Node *) newexpr;
                                        }
 
                                }
 
-                               elog(ERROR, "Function '%s' has bad return type %d",
-                                        funcname, argtype);
                                break;
                        }
                case T_Param: