NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit38fc547

committed

Re-fix wrong costing of Sort under Gather Merge.

Commitdc02c7b changed this callto create_sort_path() to take -1 rather than limit_tuples because,at that time, there was no way for a Sort beneath a Gather Mergeto become a top-N sort.Later, commit3452dc5 provideda way for a Sort beneath a Gather Merge to become a top-N sort,but failed to revert the previous commit in the process. Do that.Report and analysis by Jeff Janes; patch by Thomas Munro; review byAmit Kapila and by me.Discussion:http://postgr.es/m/CAEepm=1BWtC34vUroA0Uqjw02MaqdUrW+d6WD85_k8SLyPiKHQ@mail.gmail.com

1 parent09a65f5 commit38fc547Copy full SHA for 38fc547

File tree

1 file changed

-1

lines changed

src/backend/optimizer/plan
- planner.c

1 file changed

-1

lines changed

`‎src/backend/optimizer/plan/planner.c`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -5059,7 +5059,7 @@ create_ordered_paths(PlannerInfo *root,`
`5059`	`5059`	`ordered_rel,`
`5060`	`5060`	`cheapest_partial_path,`
`5061`	`5061`	`root->sort_pathkeys,`
`5062`		`--1.0);`
	`5062`	`+limit_tuples);`
`5063`	`5063`
`5064`	`5064`	`total_groups=cheapest_partial_path->rows*`
`5065`	`5065`	`cheapest_partial_path->parallel_workers;`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit38fc547

File tree

1 file changed

1 file changed

`‎src/backend/optimizer/plan/planner.c`

0 commit comments