database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-07-28 23:42:10 +03:00
Code Activity

Remove bgw_sighup and bgw_sigterm.

Browse Source 
Per discussion on pgsql-hackers, these aren't really needed.  Interim
versions of the background worker patch had the worker starting with
signals already unblocked, which would have made this necessary.
But the final version does not, so we don't really need it; and it
doesn't work well with the new facility for starting dynamic background
workers, so just rip it out.

Also per discussion on pgsql-hackers, back-patch this change to 9.3.
It's best to get the API break out of the way before we do an
official release of this facility, to avoid more pain for extension
authors later.
This commit is contained in:
Robert Haas
2013-07-22 14:13:00 -04:00
parent 0518eceec3
commit f40a318eea
5 changed files with 3 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
contrib/worker_spi/worker_spi.c
View File

@ -344,8 +344,6 @@ _PG_init(void)
worker.bgw_start_time = BgWorkerStart_RecoveryFinished;
worker.bgw_restart_time = BGW_NEVER_RESTART;
worker.bgw_main = worker_spi_main;
worker.bgw_sighup = NULL;
worker.bgw_sigterm = NULL;
/*
* Now fill in worker-specific data, and do the actual registrations.
@ -375,8 +373,6 @@ worker_spi_launch(PG_FUNCTION_ARGS)
worker.bgw_main = NULL; /* new worker might not have library loaded */
sprintf(worker.bgw_library_name, "worker_spi");
sprintf(worker.bgw_function_name, "worker_spi_main");
worker.bgw_sighup = NULL; /* new worker might not have library loaded */
worker.bgw_sigterm = NULL; /* new worker might not have library loaded */
snprintf(worker.bgw_name, BGW_MAXLEN, "worker %d", i);
worker.bgw_main_arg = Int32GetDatum(i);