Re: BUG #16914: Regression test of the worker_spi fails if USE_MODULE_DB environment variable is set.

On Thu, Mar 04, 2021 at 05:14:57PM -0500, Tom Lane wrote:
> CREATE EXTENSION worker_spi;
> +SELECT set_config('worker_spi.database', current_database(), false);
> SELECT worker_spi_launch(4) IS NOT NULL;
> -- wait until the worker completes its initialization
>
> but that doesn't work because the session-local setting of the GUC won't
> be seen over in the worker process.
>
> Since the alleged point of this module is to be a prototype for
> useful background workers, I think we should think of another way.
> Maybe we need to work a bit harder on passing values from
> worker_spi_launch to the worker, so that the DB name could be a
> parameter to worker_spi_launch.

Yeah, or this is converted to a TAP test where we don't care about
the database used, but that would consume more cycles without more
coverage done. Don't we need to worry about the memory context where
bgw_main_arg is pushed into for dynamic workers?
--
Michael