diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2022-07-27 12:00:10 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2022-07-27 12:00:10 -0400 |
| commit | 03361a368e7bf909283cc7721af004317fdabd3d (patch) | |
| tree | ea570b32477f0c085790100d2232363b68db8fcc /doc/src | |
| parent | a2e97cb2b6fb64c3ca3198f5c5f31190bc14c703 (diff) | |
| download | postgresql-03361a368e7bf909283cc7721af004317fdabd3d.tar.gz | |
Add missing PGDLLEXPORT markings in contrib/pg_prewarm.
After commit 089480c07, it's necessary for background worker entry
points to be marked PGDLLEXPORT, else they aren't findable by
LookupBackgroundWorkerFunction(). Since pg_prewarm lacks any
regression tests, it's not surprising its worker entry points were
overlooked. (A quick search turned up no other such oversights.)
I added some documentation pointing out the need for this, too.
Robins Tharakan and Tom Lane
CAEP4nAzndnQv3-1QKb=D-hLoK3Rko12HHMFHHtdj2GQAUXO3gw@mail.gmail.com
Diffstat (limited to 'doc/src')
| -rw-r--r-- | doc/src/sgml/bgworker.sgml | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/doc/src/sgml/bgworker.sgml b/doc/src/sgml/bgworker.sgml index 73207f72fe..3d4e4afcf9 100644 --- a/doc/src/sgml/bgworker.sgml +++ b/doc/src/sgml/bgworker.sgml @@ -141,14 +141,15 @@ typedef struct BackgroundWorker which the initial entry point for the background worker should be sought. The named library will be dynamically loaded by the worker process and <structfield>bgw_function_name</structfield> will be used to identify the - function to be called. If loading a function from the core code, this must - be set to "postgres". + function to be called. If calling a function in the core code, this must + be set to <literal>"postgres"</literal>. </para> <para> - <structfield>bgw_function_name</structfield> is the name of a function in - a dynamically loaded library which should be used as the initial entry point - for a new background worker. + <structfield>bgw_function_name</structfield> is the name of the function + to use as the initial entry point for the new background worker. If + this function is in a dynamically loaded library, it must be marked + <literal>PGDLLEXPORT</literal> (and not <literal>static</literal>). </para> <para> |