pgsql: Disable cache clobber to avoid breaking postgres_fdw termination - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Disable cache clobber to avoid breaking postgres_fdw termination
Date
Msg-id E1ldyyI-0001qi-NG@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Disable cache clobber to avoid breaking postgres_fdw termination test.

Commit 93f414614 improved a pre-existing test case so that it would
show whether or not termination of the "remote" worker process happened.
This soon exposed that, when debug_invalidate_system_caches_always
(nee CLOBBER_CACHE_ALWAYS) is enabled, no such termination occurs.
That's because cache invalidation forces postgres_fdw connections
to be dropped at end of transaction, so that there's no worker to
terminate.  There's a race condition as to whether the worker will
manage to get out of the BackendStatusArray before we look, but at
least on buildfarm member hyrax, it's failed twice in two attempts.

Rather than re-lobotomizing the test, let's fix this by transiently
disabling debug_invalidate_system_caches_always.  (Hooray for that
being just a GUC nowadays, rather than a compile-time option.)
If this proves not to be enough to make the test stable, we can
do the other thing instead.

Discussion: https://postgr.es/m/3854538.1620081771@sss.pgh.pa.us

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/1273a15bf91fa322915e32d3b6dc6ec916397268

Modified Files
--------------
contrib/postgres_fdw/expected/postgres_fdw.out | 13 ++++++++++---
contrib/postgres_fdw/sql/postgres_fdw.sql      | 16 +++++++++++++---
2 files changed, 23 insertions(+), 6 deletions(-)


pgsql-committers by date:

Previous
From: Alvaro Herrera
Date:
Subject: pgsql: Fix OID passed to object-alter hook during ALTER CONSTRAINT
Next
From: Peter Eisentraut
Date:
Subject: pgsql: GUC description improvements for clarity