NotificationsYou must be signed in to change notification settings
Fork28
Star153

Commit74618e7

committed

Handle lack of DSM slots in parallel btree build.

If no DSM slots are available, a ParallelContext can still becreated, but its seg pointer is NULL. Teach parallel btree buildto cope with that by falling back to a regular non-parallel build,to avoid crashing with a segmentation fault.Back-patch to 11, where parallel CREATE INDEX landed.Reported-by: Nicola ContuReviewed-by: Peter GeogheganDiscussion:https://postgr.es/m/CA%2BhUKGJgJEBnkuODBVomyK3MWFvDBbMVj%3Dgdt6DnRPU-5sQ6UQ%40mail.gmail.com

1 parentc9d2977 commit74618e7Copy full SHA for 74618e7

File tree

1 file changed

-0

lines changed

src/backend/access/nbtree
- nbtsort.c

1 file changed

-0

lines changed

`‎src/backend/access/nbtree/nbtsort.c‎`

Lines changed: 9 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1331,6 +1331,15 @@ _bt_begin_parallel(BTBuildState *buildstate, bool isconcurrent, int request)`
`1331`	`1331`	`Assert(request>0);`
`1332`	`1332`	`pcxt=CreateParallelContext("postgres","_bt_parallel_build_main",`
`1333`	`1333`	`request);`
	`1334`	`+`
	`1335`	`+/* If no DSM segment was available, back out (do serial build) */`
	`1336`	`+if (pcxt->seg==NULL)`
	`1337`	`+{`
	`1338`	`+DestroyParallelContext(pcxt);`
	`1339`	`+ExitParallelMode();`
	`1340`	`+return;`
	`1341`	`+}`
	`1342`	`+`
`1334`	`1343`	`scantuplesortstates=leaderparticipates ?request+1 :request;`
`1335`	`1344`
`1336`	`1345`	`/*`

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!

Commit74618e7

File tree

1 file changed

1 file changed

`‎src/backend/access/nbtree/nbtsort.c‎`

0 commit comments