NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit2c56b3a

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 parenta5b652f commit2c56b3aCopy full SHA for 2c56b3a

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
`@@ -1256,6 +1256,15 @@ _bt_begin_parallel(BTBuildState *buildstate, bool isconcurrent, int request)`
`1256`	`1256`	`Assert(request>0);`
`1257`	`1257`	`pcxt=CreateParallelContext("postgres","_bt_parallel_build_main",`
`1258`	`1258`	`request, true);`
	`1259`	`+`
	`1260`	`+/* If no DSM segment was available, back out (do serial build) */`
	`1261`	`+if (pcxt->seg==NULL)`
	`1262`	`+{`
	`1263`	`+DestroyParallelContext(pcxt);`
	`1264`	`+ExitParallelMode();`
	`1265`	`+return;`
	`1266`	`+}`
	`1267`	`+`
`1259`	`1268`	`scantuplesortstates=leaderparticipates ?request+1 :request;`
`1260`	`1269`
`1261`	`1270`	`/*`

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!

Commit2c56b3a

File tree

1 file changed

1 file changed

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

0 commit comments