From 8964b41c7b88bb1acc7bb9811ca8cb4938c7a410 Mon Sep 17 00:00:00 2001 From: Neil Conway Date: Wed, 8 Nov 2006 00:45:30 +0000 Subject: [PATCH] Remove a 15-year old comment questioning behavior that is now well- established: referencing an undefined parameter should result in an error, not NULL. --- src/backend/executor/execQual.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/backend/executor/execQual.c b/src/backend/executor/execQual.c index 6559947bf1..a827335c62 100644 --- a/src/backend/executor/execQual.c +++ b/src/backend/executor/execQual.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/executor/execQual.c,v 1.197 2006/11/06 18:21:31 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/executor/execQual.c,v 1.198 2006/11/08 00:45:30 neilc Exp $ * *------------------------------------------------------------------------- */ @@ -595,11 +595,6 @@ ExecEvalConst(ExprState *exprstate, ExprContext *econtext, * something like ($.name) and the expression context contains * the current parameter bindings (name = "sam") (age = 34)... * so our job is to find and return the appropriate datum ("sam"). - * - * Q: if we have a parameter ($.foo) without a binding, i.e. - * there is no (foo = xxx) in the parameter list info, - * is this a fatal error or should this be a "not available" - * (in which case we could return NULL)? -cim 10/13/89 * ---------------------------------------------------------------- */ static Datum -- 2.11.0