Fix comments about bgworker registration before MaxBackends initialization
authorMichael Paquier <michael@paquier.xyz>
Sat, 29 Jan 2022 01:47:36 +0000 (10:47 +0900)
committerMichael Paquier <michael@paquier.xyz>
Sat, 29 Jan 2022 01:47:36 +0000 (10:47 +0900)
commit5ecd0183fb6afa4a07aad71ea0e08c70f64a42a9
treed5e8223c09a9a85c7412a521a3a7d1dd096f9627
parentbf42fcace5c510b3579469913ae4cbb2517861d5
Fix comments about bgworker registration before MaxBackends initialization

Since 6bc8ef0b, InitializeMaxBackends() has used max_worker_processes
instead of adapting MaxBackends to the number of background workers
registered by modules loaded in shared_preload_libraries (at this time,
bgworkers were only static, but gained dynamic capabilities as a matter
of supporting parallel queries meaning that a control cap was
necessary).

Some comments referred to the past registration logic, making them
confusing and incorrect, so fix these.

Some of the out-of-core modules that could be loaded in this path
sometimes like to manipulate dynamically some of the resource-related
GUCs for their own needs, this commit adds a note about that.

Author: Nathan Bossart
Discussion: https://postgr.es/m/20220127181815.GA551692@nathanxps13
src/backend/postmaster/postmaster.c
src/backend/utils/init/postinit.c