NotificationsYou must be signed in to change notification settings
Fork5k
Star18.1k

Commit7d11f36

committed

aio: Fix configuration reload in IO workers.

method_worker.c installed SignalHandlerForConfigReload, but it failed toactually process reload requests. That hasn't yet produced any concreteproblem reports in terms of GUC changes it should have cared about inv18, but it was inconsistent.It did cause problems for a couple of patches in development that needIO workers to react to ALTER SYSTEM + pg_reload_conf(). Fix extractedfrom one of those patches.Back-patch to 18.Reported-by: Dmitry Dolgov <9erthalion6@gmail.com>Discussion:https://postgr.es/m/sh5uqe4a4aqo5zkkpfy5fobe2rg2zzouctdjz7kou4t74c66ql%40yzpkxb7pgoxf

1 parentb4c19da commit7d11f36Copy full SHA for 7d11f36

File tree

1 file changed

-0

lines changed

src/backend/storage/aio
- method_worker.c

1 file changed

-0

lines changed

`‎src/backend/storage/aio/method_worker.c`

Lines changed: 6 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -575,6 +575,12 @@ IoWorkerMain(const void *startup_data, size_t startup_data_len)`
`575`	`575`	`}`
`576`	`576`
`577`	`577`	`CHECK_FOR_INTERRUPTS();`
	`578`	`+`
	`579`	`+if (ConfigReloadPending)`
	`580`	`+{`
	`581`	`+ConfigReloadPending= false;`
	`582`	`+ProcessConfigFile(PGC_SIGHUP);`
	`583`	`+}`
`578`	`584`	`}`
`579`	`585`
`580`	`586`	`error_context_stack=errcallback.previous;`

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!

Commit7d11f36

File tree

1 file changed

1 file changed

`‎src/backend/storage/aio/method_worker.c`

0 commit comments