NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit2c6e647

committed

Allow treating TABLESAMPLE scans as parallel-safe.

This was the intention all along, but an extraneous "return;" inset_rel_consider_parallel() caused sampled rels to never be markedconsider_parallel.Since we don't have any partial tablesample path/plan type yet, there'sno possibility of parallelizing the sample scan itself; but this fixallows such a scan to appear below a parallel join, for example.

1 parent0e495c5 commit2c6e647Copy full SHA for 2c6e647

File tree

1 file changed

-1

lines changed

src/backend/optimizer/path
- allpaths.c

1 file changed

-1

lines changed

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

Lines changed: 0 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -538,7 +538,6 @@ set_rel_consider_parallel(PlannerInfo root, RelOptInfo rel,`
`538`	`538`	`if (has_parallel_hazard((Node*)rte->tablesample->args,`
`539`	`539`	`false))`
`540`	`540`	`return;`
`541`		`-return;`
`542`	`541`	`}`
`543`	`542`
`544`	`543`	`/*`

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!

Commit2c6e647

File tree

1 file changed

1 file changed

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

0 commit comments