NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commita8af856

Etsuro Fujita

committed

Adjust input value to WaitEventSetWait() in ExecAppendAsyncEventWait().

Adjust the number of events given to WaitEventSetWait() so that itdoesn't exceed the maximum number of events in the WaitEventSet givento that function (set->nevents_space) in hopes of making the buildfarmgreen.Per valgrind failure report from Tom Lane and the buildfarm.Author: Etsuro FujitaDiscussion:https://postgr.es/m/3411577.1617289776%40sss.pgh.pa.us

1 parent82ed774 commita8af856Copy full SHA for a8af856

File tree

1 file changed

-1

lines changed

src/backend/executor
- nodeAppend.c

1 file changed

-1

lines changed

`‎src/backend/executor/nodeAppend.c`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1001,6 +1001,7 @@ ExecAppendAsyncEventWait(AppendState *node)`
`1001`	`1001`	`longtimeout=node->as_syncdone ?-1 :0;`
`1002`	`1002`	`WaitEventoccurred_event[EVENT_BUFFER_SIZE];`
`1003`	`1003`	`intnoccurred;`
	`1004`	`+intnevents;`
`1004`	`1005`	`inti;`
`1005`	`1006`
`1006`	`1007`	`/* We should never be called when there are no valid async subplans. */`
`@@ -1022,8 +1023,9 @@ ExecAppendAsyncEventWait(AppendState *node)`
`1022`	`1023`	`}`
`1023`	`1024`
`1024`	`1025`	`/* Wait for at least one event to occur. */`
	`1026`	`+nevents=Min(node->as_nasyncplans+1,EVENT_BUFFER_SIZE);`
`1025`	`1027`	`noccurred=WaitEventSetWait(node->as_eventset,timeout,occurred_event,`
`1026`		`-EVENT_BUFFER_SIZE,WAIT_EVENT_APPEND_READY);`
	`1028`	`+nevents,WAIT_EVENT_APPEND_READY);`
`1027`	`1029`	`FreeWaitEventSet(node->as_eventset);`
`1028`	`1030`	`node->as_eventset=NULL;`
`1029`	`1031`	`if (noccurred==0)`

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!

Commita8af856

File tree

1 file changed

1 file changed

`‎src/backend/executor/nodeAppend.c`

0 commit comments