From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Mon, 25 Apr 2005 04:27:12 +0000 (+0000)
Subject: Avoid rechecking lossy operators twice in a bitmap scan plan.
X-Git-Tag: REL8_1_0BETA1~917
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a0ea71333a800f647179ff120f7bdee30b361ed5;p=postgresql

Avoid rechecking lossy operators twice in a bitmap scan plan.
---

diff --git a/src/backend/optimizer/plan/createplan.c b/src/backend/optimizer/plan/createplan.c
index 3feff40423..e042468c66 100644
--- 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,