NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commit65a603e

committed

Guard against parallel-restricted functions in VALUES expressions.

Obvious brain fade in set_rel_consider_parallel(). Noticed it whileadjusting the adjacent RTE_FUNCTION case.In 9.6, also make the code look more like what I just did in HEADby removing the unnecessary function_rte_parallel_ok subroutine(it does nothing that expression_tree_walker wouldn't do).

1 parentda1c916 commit65a603eCopy full SHA for 65a603e

File tree

1 file changed

-5

lines changed

src/backend/optimizer/path
- allpaths.c

1 file changed

-5

lines changed

`‎src/backend/optimizer/path/allpaths.c`

Lines changed: 3 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -599,11 +599,9 @@ set_rel_consider_parallel(PlannerInfo root, RelOptInfo rel,`
`599`	`599`	`break;`
`600`	`600`
`601`	`601`	`caseRTE_VALUES:`
`602`		`-`
`603`		`-/*`
`604`		`- * The data for a VALUES clause is stored in the plan tree itself,`
`605`		`- * so scanning it in a worker is fine.`
`606`		`- */`
	`602`	`+/* Check for parallel-restricted functions. */`
	`603`	`+if (!is_parallel_safe(root, (Node*)rte->values_lists))`
	`604`	`+return;`
`607`	`605`	`break;`
`608`	`606`
`609`	`607`	`caseRTE_CTE:`

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!

Commit65a603e

File tree

1 file changed

1 file changed

`‎src/backend/optimizer/path/allpaths.c`

0 commit comments