diff options
author | Michael Paquier <michael@paquier.xyz> | 2023-07-21 12:07:52 +0900 |
---|---|---|
committer | Michael Paquier <michael@paquier.xyz> | 2023-07-21 12:07:52 +0900 |
commit | 97ff8dd02ca788020021cdafb85d77d4fd3f3125 (patch) | |
tree | e98aedfa0fc696cfd94a34b0c5c460807b133ad5 /src/backend/nodes/nodeFuncs.c | |
parent | 9089287aa037fdecb5a52cec1926e5ae9569e9f9 (diff) | |
download | postgresql-97ff8dd02ca788020021cdafb85d77d4fd3f3125.tar.gz postgresql-97ff8dd02ca788020021cdafb85d77d4fd3f3125.zip |
Fix worker_spi when launching workers without shared_preload_libraries
Currently, the database name to connect is initialized only when the
module is loaded with shared_preload_libraries, causing any call of
worker_spi_launch() to fail if the library is not loaded for a dynamic
bgworker launch. Rather than making the GUC defining the database to
connect to a PGC_POSTMASTER, this commit switches worker_spi.database to
PGC_SIGHUP, loaded even if the module's library is loaded dynamically
for a worker.
We have been discussing about the integration of more advanced tests in
this module, with and without shared_preload_libraries set, so this
eases a bit the work planned in this area.
No backpatch is done as, while this is a bug, it changes the definition
of worker_spi.database.
Author: Masahiro Ikeda
Reviewed-by: Bharath Rupireddy
Discussion: https://postgr.es/m/d30d3ea7d21cb7c9e1e3cc47e301f1b6@oss.nttdata.com
Diffstat (limited to 'src/backend/nodes/nodeFuncs.c')
0 files changed, 0 insertions, 0 deletions