Commitdc02c7b

committed

Fix wrong costing of Sort under Gather Merge.

There's no mechanism for such a sort to become a top-N sort, so weshould pass -1 rather than limit_tuples to cost_sort().Rushabh Lathia, per a report from Mithun CyDiscussion:http://postgr.es/m/CAGPqQf1akRcSgC9=6iwx=sEPap9UvPpHJLzg8_N+OuHdb6fL+g@mail.gmail.com

1 parent6b76f1b commitdc02c7bCopy full SHA for dc02c7b

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -4416,7 +4416,7 @@ create_ordered_paths(PlannerInfo *root,`
`4416`	`4416`	`ordered_rel,`
`4417`	`4417`	`cheapest_partial_path,`
`4418`	`4418`	`root->sort_pathkeys,`
`4419`		`-limit_tuples);`
	`4419`	`+-1.0);`
`4420`	`4420`
`4421`	`4421`	`total_groups=cheapest_partial_path->rows*`
`4422`	`4422`	`cheapest_partial_path->parallel_workers;`

Comments

(0)