pgsql: Revert the commits related to allowing page lock to conflict amo - Mailing list pgsql-committers

From Amit Kapila
Subject pgsql: Revert the commits related to allowing page lock to conflict amo
Date
Msg-id E1qHFlb-0028Bh-7W@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Revert the commits related to allowing page lock to conflict among parallel group members.

This commit reverts the work done by commits 3ba59ccc89 and 72e78d831a.
Those commits were incorrect in asserting that we never acquire any other
heavy-weight lock after acquring page lock other than relation extension
lock. We can acquire a lock on catalogs while doing catalog look up after
acquring page lock.

This won't impact any existing feature but we need to think some other way
to achieve this before parallelizing other write operations or even
improving the parallelism in vacuum (like allowing multiple workers
for an index).

Reported-by: Jaime Casanova
Author: Amit Kapila
Backpatch-through: 13
Discussion: https://postgr.es/m/CAJKUy5jffnRKNvRHKQ0LynRb0RJC-o4P8Ku3x9vGAVLwDBWumQ@mail.gmail.com

Branch
------
REL_15_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/907d3dd53159fb50402ec292ad60b2452c281310

Modified Files
--------------
src/backend/optimizer/plan/planner.c | 12 +++++++-----
src/backend/storage/lmgr/README      | 34 +++++++++++++---------------------
src/backend/storage/lmgr/deadlock.c  |  9 ++++-----
src/backend/storage/lmgr/lock.c      | 32 ++++----------------------------
src/backend/storage/lmgr/proc.c      | 12 ++++++------
5 files changed, 34 insertions(+), 65 deletions(-)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: Handle \v as a whitespace character in parsers
Next
From: Michael Paquier
Date:
Subject: pgsql: Add newline at the end of header generated by generate-wait_even