pgsql: Fix failure with lock mode used for custom relation options - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Fix failure with lock mode used for custom relation options
Date
Msg-id E1iCvqJ-0000SG-4M@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix failure with lock mode used for custom relation options

In-core relation options can use a custom lock mode since 47167b7, that
has lowered the lock available for some autovacuum parameters.  However
it forgot to consider custom relation options.  This causes failures
with ALTER TABLE SET when changing a custom relation option, as its lock
is not defined.  The existing APIs to define a custom reloption does not
allow to define a custom lock mode, so enforce its initialization to
AccessExclusiveMode which should be safe enough in all cases.  An
upcoming patch will extend the existing APIs to allow a custom lock mode
to be defined.

The problem can be reproduced with bloom indexes, so add a test there.

Reported-by: Nikolay Sharplov
Analyzed-by: Thomas Munro, Michael Paquier
Author: Michael Paquier
Reviewed-by: Kuntal Ghosh
Discussion: https://postgr.es/m/20190920013831.GD1844@paquier.xyz
Backpatch-through: 9.6

Branch
------
REL_11_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/d01d4f23742bb7bfedeb47f16445c3478a9cd3d8

Modified Files
--------------
contrib/bloom/expected/bloom.out       | 1 +
contrib/bloom/sql/bloom.sql            | 1 +
src/backend/access/common/reloptions.c | 7 +++++++
3 files changed, 9 insertions(+)


pgsql-committers by date:

Previous
From: Alexander Korotkov
Date:
Subject: pgsql: Fix bug in pairingheap_SpGistSearchItem_cmp()
Next
From: Michael Paquier
Date:
Subject: pgsql: Allow definition of lock mode for custom reloptions