NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitb5afdde

Etsuro Fujita

committed

Fix testing of parallel-safety of scan/join target.

In commit960df2a ("Correctly assess parallel-safety of tlists whenSRFs are used."), the testing of scan/join target was done incorrectly,which caused a plan-quality problem. Backpatch through to v11 wherethe aforementioned commit went in, since this is a regression from v10.Author: Etsuro FujitaReviewed-by: Robert Haas and Tom LaneDiscussion:https://postgr.es/m/5C75303E.8020303@lab.ntt.co.jp

1 parent6f91815 commitb5afddeCopy full SHA for b5afdde

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
`@@ -1993,7 +1993,7 @@ grouping_planner(PlannerInfo *root, bool inheritance_update,`
`1993`	`1993`	`{`
`1994`	`1994`	`scanjoin_target=make_group_input_target(root,final_target);`
`1995`	`1995`	`scanjoin_target_parallel_safe=`
`1996`		`-is_parallel_safe(root, (Node*)grouping_target->exprs);`
	`1996`	`+is_parallel_safe(root, (Node*)scanjoin_target->exprs);`
`1997`	`1997`	`}`
`1998`	`1998`	`else`
`1999`	`1999`	`{`

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!

Commitb5afdde

File tree

1 file changed

1 file changed

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

0 commit comments