Planner did the wrong thing with index-scan-backward plans: generated

tglsfdc · tglsfdc · commit 9191d684a7da · 2000-07-13T05:47:29.000Z
them, but forgot to attach relevant restriction clauses, so that the
plan represented a scan over the whole table with restrictions applied
as qpquals not indexquals.  Another day, another bug...
diff --git a/src/backend/optimizer/path/indxpath.c b/src/backend/optimizer/path/indxpath.c
@@ -9,7 +9,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/optimizer/path/indxpath.c,v 1.86 2000/07/05 23:11:22 tgl Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/optimizer/path/indxpath.c,v 1.87 2000/07/13 05:47:29 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -195,7 +195,7 @@ create_index_paths(Query *root,
 			 useful_for_ordering(root, rel, index, ForwardScanDirection))
 				add_path(rel, (Path *)
 						 create_index_path(root, rel, index,
-										   NIL,
+										   restrictclauses,
 										   ForwardScanDirection));
 		}
 
@@ -207,7 +207,7 @@ create_index_paths(Query *root,
 		if (useful_for_ordering(root, rel, index, BackwardScanDirection))
 			add_path(rel, (Path *)
 					 create_index_path(root, rel, index,
-									   NIL,
+									   restrictclauses,
 									   BackwardScanDirection));
 
 		/*

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`*`
`10`	`10`	`*`
`11`	`11`	`* IDENTIFICATION`
`12`		`- * $Header: /cvsroot/pgsql/src/backend/optimizer/path/indxpath.c,v 1.86 2000/07/05 23:11:22 tgl Exp $`
	`12`	`+ * $Header: /cvsroot/pgsql/src/backend/optimizer/path/indxpath.c,v 1.87 2000/07/13 05:47:29 tgl Exp $`
`13`	`13`	`*`
`14`	`14`	`*-------------------------------------------------------------------------`
`15`	`15`	`*/`
`@@ -195,7 +195,7 @@ create_index_paths(Query *root,`
`195`	`195`	`useful_for_ordering(root, rel, index, ForwardScanDirection))`
`196`	`196`	`add_path(rel, (Path *)`
`197`	`197`	`create_index_path(root, rel, index,`
`198`		`- NIL,`
	`198`	`+ restrictclauses,`
`199`	`199`	`ForwardScanDirection));`
`200`	`200`	`}`
`201`	`201`
`@@ -207,7 +207,7 @@ create_index_paths(Query *root,`
`207`	`207`	`if (useful_for_ordering(root, rel, index, BackwardScanDirection))`
`208`	`208`	`add_path(rel, (Path *)`
`209`	`209`	`create_index_path(root, rel, index,`
`210`		`- NIL,`
	`210`	`+ restrictclauses,`
`211`	`211`	`BackwardScanDirection));`
`212`	`212`
`213`	`213`	`/*`