> > So, the patch for "function() BETWEEN" helps, but I'm still not sure
> > it is safe, since I don't know how these nodes might be handled
> > deeper in the backend.
>
> Ship the patch over here and I will have a look at it.
OK, here it is...
- Tom
*** ../src/backend/parser/parse_expr.c.orig Sun Mar 1 15:04:42 1998
--- ../src/backend/parser/parse_expr.c Sun Mar 15 05:49:03 1998
***************
*** 301,306 ****
--- 301,321 ----
result = (Node *) expr;
break;
}
+ /* These nodes do _not_ come from the original parse tree.
+ * They result from parser transformation in this phase.
+ * At least one construct (BETWEEN/AND) puts the same nodes
+ * into two branches of the parse tree. Hence, some nodes
+ * are transformed twice. These nodes come from transforming
+ * a function call. Let's try just passing them through...
+ * - thomas 1998-03-14
+ */
+ case T_Expr:
+ case T_Var:
+ case T_Const:
+ {
+ result = (Node *) expr;
+ break;
+ }
default:
/* should not reach here */
elog(ERROR, "transformExpr: does not know how to transform node %d",