From f538329f9db21e32220d73e77cc851573bfb49f7 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Mon, 3 Dec 2007 22:37:17 +0000 Subject: [PATCH] Fix build_minmax_path() to cope if an IS NULL clause turns up in the indexable-clauses list for a btree index. Formerly it just Asserted that all such clauses were opclauses, but that's no longer true in 8.3. Per bug #3796 from Matthias Schoeneich. --- src/backend/optimizer/plan/planagg.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/backend/optimizer/plan/planagg.c b/src/backend/optimizer/plan/planagg.c index 09302d3fc1d..11f15591cbd 100644 --- a/src/backend/optimizer/plan/planagg.c +++ b/src/backend/optimizer/plan/planagg.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/optimizer/plan/planagg.c,v 1.34 2007/11/15 21:14:36 momjian Exp $ + * $PostgreSQL: pgsql/src/backend/optimizer/plan/planagg.c,v 1.35 2007/12/03 22:37:17 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -368,7 +368,9 @@ build_minmax_path(PlannerInfo *root, RelOptInfo *rel, MinMaxAggInfo *info) RestrictInfo *rinfo = (RestrictInfo *) lfirst(ll); int strategy; - Assert(is_opclause(rinfo->clause)); + /* Could be an IS_NULL test, if so ignore */ + if (!is_opclause(rinfo->clause)) + continue; strategy = get_op_opfamily_strategy(((OpExpr *) rinfo->clause)->opno, index->opfamily[prevcol]);