Commitdf79b84

committed

genericcostestimate() neglected to include qual startup cost in

indexTotalCost. I think this may not make any real difference in 7.4,but it definitely is a problem with CVS tip's new equation.

1 parentf5d8f0b commitdf79b84Copy full SHA for df79b84

File tree

-1

lines changed

-1

lines changed

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@`
`15`	`15`	`*`
`16`	`16`	`*`
`17`	`17`	`* IDENTIFICATION`
`18`		`- * $PostgreSQL: pgsql/src/backend/utils/adt/selfuncs.c,v 1.157 2004/02/17 00:52:53 tgl Exp $`
	`18`	`+ * $PostgreSQL: pgsql/src/backend/utils/adt/selfuncs.c,v 1.158 2004/02/27 21:44:34 tgl Exp $`
`19`	`19`	`*`
`20`	`20`	`*-------------------------------------------------------------------------`
`21`	`21`	`*/`
`@@ -4259,6 +4259,7 @@ genericcostestimate(Query root, RelOptInfo rel,`
`4259`	`4259`	`if (qual_arg_cost<0)/* just in case... */`
`4260`	`4260`	`qual_arg_cost=0;`
`4261`	`4261`	`*indexStartupCost=qual_arg_cost;`
	`4262`	`+*indexTotalCost+=qual_arg_cost;`
`4262`	`4263`	`indexTotalCost+=numIndexTuples (cpu_index_tuple_cost+qual_op_cost);`
`4263`	`4264`
`4264`	`4265`	`/*`

Comments

(0)