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.

diff --git a/src/backend/optimizer/plan/planagg.c b/src/backend/optimizer/plan/planagg.c
index 09302d3fc1db446e107f87c01d0a03f70f732178..11f15591cbd3a533ba65c48aed90b4a850c855ce 100644 (file)
--- 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]);