NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitf24cf96

committed

Fix test for subplans in force-parallel mode.

We mustn't force parallel mode if the query has any subplans, sinceExecSerializePlan doesn't transmit them to workers. Testingtop_plan->initPlan is inadequate because (1) there might be initPlansattached to lower plan nodes, and (2) non-initPlan subplans don'twork either. There's certainly room for improvement in thoserestrictions, but for the moment that's what we've got.Amit Kapila, per report from Andreas SeltenreichDiscussion: <8737im6pmh.fsf@credativ.de>

1 parentc5f365f commitf24cf96Copy full SHA for f24cf96

File tree

1 file changed

-2

lines changed

src/backend/optimizer/plan
- planner.c

1 file changed

-2

lines changed

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

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -342,11 +342,11 @@ standard_planner(Query *parse, int cursorOptions, ParamListInfo boundParams)`
`342`	`342`	`* actually a safe thing to do. (Note: we assume adding a Material node`
`343`	`343`	`* above did not change the parallel safety of the plan, so we can still`
`344`	`344`	`* rely on best_path->parallel_safe. However, that flag doesn't account`
`345`		`- * forinitPlans, whichrender the plan parallel-unsafe.)`
	`345`	`+ * forsubplans, whichwe are unable to transmit to workers presently.)`
`346`	`346`	`*/`
`347`	`347`	`if (force_parallel_mode!=FORCE_PARALLEL_OFF&&`
`348`	`348`	`best_path->parallel_safe&&`
`349`		`-top_plan->initPlan==NIL)`
	`349`	`+glob->subplans==NIL)`
`350`	`350`	`{`
`351`	`351`	`Gather*gather=makeNode(Gather);`
`352`	`352`

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!

Commitf24cf96

File tree

1 file changed

1 file changed

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

0 commit comments