Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit4922173

Browse files
committed
worker_spi: Fix test failure with BGWORKER_BYPASS_ROLELOGINCHECK
This is a consequence of4817da5 that has bumped upmax_worker_processes, where now the last worker started by the testwould be able to start by itself a parallel worker because there aremore slots available. This did not show up before as the number ofbgworkers reached exactly 8, as known as the previous limit, at the endof the test.Per report from buildfarm member crake, reproducible withdebug_parallel_query = regress in the same fashion asfd4d93d.
1 parent63a5822 commit4922173

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

‎src/test/modules/worker_spi/worker_spi.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -174,11 +174,11 @@ worker_spi_main(Datum main_arg)
174174
worker_spi_role,flags);
175175

176176
/*
177-
* Disable parallel query for workers started with BYPASS_ALLOWCONNso as
178-
* these don't attempt connectionsto the database that may not allow
179-
* that.
177+
* Disable parallel query for workers started with BYPASS_ALLOWCONNor
178+
*BGWORKER_BYPASS_ALLOWCONN so asthese don't attempt connectionsusing a
179+
*database or a role that may not allowthat.
180180
*/
181-
if (flags&BGWORKER_BYPASS_ALLOWCONN)
181+
if ((flags&(BGWORKER_BYPASS_ALLOWCONN |BGWORKER_BYPASS_ROLELOGINCHECK)))
182182
SetConfigOption("max_parallel_workers_per_gather","0",
183183
PGC_USERSET,PGC_S_OVERRIDE);
184184

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp