aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2007-12-03 22:37:17 +0000
committerTom Lane <tgl@sss.pgh.pa.us>2007-12-03 22:37:17 +0000
commitf538329f9db21e32220d73e77cc851573bfb49f7 (patch)
tree9ddc4822b7cbd9e1dd7a60d28f611f181642b33a /src
parent50eed107ae63dc503fc58bc1a2e679ce232e283c (diff)
downloadpostgresql-f538329f9db21e32220d73e77cc851573bfb49f7.tar.gz
postgresql-f538329f9db21e32220d73e77cc851573bfb49f7.zip
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.
Diffstat (limited to 'src')
-rw-r--r--src/backend/optimizer/plan/planagg.c6
1 files 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]);