pgsql: test_custom_stats: Fail if loading module outside shared_preload - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: test_custom_stats: Fail if loading module outside shared_preload
Date
Msg-id E1wfBIa-0013FO-0D@gemulon.postgresql.org
Whole thread
List pgsql-committers
test_custom_stats: Fail if loading module outside shared_preload_libraries

Previously, test_custom_var_stats and test_custom_fixed_stats silently
skipped pgstat_register_kind() when not loaded via
shared_preload_libraries, behavior inherited from injection_points.
This left the SQL functions callable without the kind registered,
leading to various issues on the backend side.

This code is not designed to work without the pgstats kinds registered.
pgstat_register_kind() gets now called when these libraries are loaded,
with or without shared_preload_libraries, letting the registration fail
if loading the modules at a later step than startup.  test_custom_rmgrs
does the same thing.

Author: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
Reviewed-by: Ewan Young <kdbase.hack@gmail.com>
Discussion: https://postgr.es/m/akS/ldidWeqG1FWk@bdtpg
Backpatch-through: 19

Branch
------
REL_19_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/7838efe9a2d165cc3cf3a4538ecc5f10d49b1a49

Modified Files
--------------
src/test/modules/test_custom_stats/test_custom_fixed_stats.c | 4 ----
src/test/modules/test_custom_stats/test_custom_var_stats.c   | 4 ----
2 files changed, 8 deletions(-)


pgsql-committers by date:

Previous
From: John Naylor
Date:
Subject: pgsql: Fix loss of precision in pg_stat_us_to_ms()
Next
From: John Naylor
Date:
Subject: pgsql: Use ssup_datum_*_cmp in more places