Avoid rechecking lossy operators twice in a bitmap scan plan.

tglsfdc · tglsfdc · commit a0ea71333a80 · 2005-04-25T04:27:12.000Z
diff --git a/src/backend/optimizer/plan/createplan.c b/src/backend/optimizer/plan/createplan.c
@@ -10,7 +10,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/optimizer/plan/createplan.c,v 1.187 2005/04/25 03:58:30 tgl Exp $
+ *	  $PostgreSQL: pgsql/src/backend/optimizer/plan/createplan.c,v 1.188 2005/04/25 04:27:12 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -889,6 +889,14 @@ create_bitmap_scan_plan(Query *root,
 	/* Sort clauses into best execution order */
 	qpqual = order_qual_clauses(root, qpqual);
 
+	/*
+	 * When dealing with special or lossy operators, we will at this point
+	 * have duplicate clauses in qpqual and bitmapqualorig.  We may as well
+	 * drop 'em from bitmapqualorig, since there's no point in making the
+	 * tests twice.
+	 */
+	bitmapqualorig = list_difference_ptr(bitmapqualorig, qpqual);
+
 	/* Finally ready to build the plan node */
 	scan_plan = make_bitmap_heapscan(tlist,
 									 qpqual,
