Restructure representation of aggregate functions so that they have pg_proc

entries, per pghackers discussion.  This fixes aggregates to live in
namespaces, and also simplifies/speeds up lookup in parse_func.c.
Also, add a 'proimplicit' flag to pg_proc that controls whether a type
coercion function may be invoked implicitly, or only explicitly.  The
current settings of these flags are more permissive than I would like,
but we will need to debate and refine the behavior; for now, I avoided
breaking regression tests as much as I could.

1 files changed, 6 insertions, 2 deletions

diff --git a/src/include/parser/parse_func.h b/src/include/parser/parse_func.h
index 3a7641e1de7..e6353be34e7 100644
--- a/src/include/parser/parse_func.h
+++ b/src/include/parser/parse_func.h
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2001, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: parse_func.h,v 1.38 2002/04/09 20:35:55 tgl Exp $
+ * $Id: parse_func.h,v 1.39 2002/04/11 20:00:15 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -42,7 +42,8 @@ typedef struct _CandidateList
 typedef enum
 {
 	FUNCDETAIL_NOTFOUND,		/* no suitable interpretation */
-	FUNCDETAIL_NORMAL,			/* found a matching function */
+	FUNCDETAIL_NORMAL,			/* found a matching regular function */
+	FUNCDETAIL_AGGREGATE,		/* found a matching aggregate function */
 	FUNCDETAIL_COERCION			/* it's a type coercion request */
 } FuncDetailCode;
 
@@ -62,6 +63,9 @@ extern void func_error(const char *caller, List *funcname,
 					   int nargs, const Oid *argtypes,
 					   const char *msg);
 
+extern Oid	find_aggregate_func(const char *caller, List *aggname,
+								Oid basetype);
+
 extern Oid	LookupFuncName(List *funcname, int nargs, const Oid *argtypes);
 extern Oid	LookupFuncNameTypeNames(List *funcname, List *argtypes,
 									bool opaqueOK, const char *caller);