pgsql: Revert "For inplace update, send nontransactional invalidations.

From: Noah Misch <noah(at)leadboat(dot)com>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Revert "For inplace update, send nontransactional invalidations.
Date: 2024-11-02 16:08:47
Message-ID: E1t7GgC-003cW6-3O@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Revert "For inplace update, send nontransactional invalidations."

This reverts commit 95c5acb3fc261067ab65ddc0b2dca8e162f09442 (v17) and
counterparts in each other non-master branch. If released, that commit
would have caused a worst-in-years minor release regression, via
undetected LWLock self-deadlock. This commit and its self-deadlock fix
warrant more bake time in the master branch.

Reported by Alexander Lakhin.

Discussion: https://postgr.es/m/10ec0bc3-5933-1189-6bb8-5dec4114558e@gmail.com

Branch
------
REL_14_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/4c708872950374dce23a61a1eba1411d9498d5d2

Modified Files
--------------
src/backend/access/heap/heapam.c | 43 +----
src/backend/access/transam/xact.c | 26 +--
src/backend/catalog/index.c | 11 +-
src/backend/replication/logical/decode.c | 26 +--
src/backend/utils/cache/catcache.c | 7 +-
src/backend/utils/cache/inval.c | 261 +++++++-------------------
src/backend/utils/cache/syscache.c | 3 +-
src/include/utils/catcache.h | 3 +-
src/include/utils/inval.h | 6 -
src/test/isolation/expected/inplace-inval.out | 10 +-
src/test/isolation/specs/inplace-inval.spec | 12 +-
11 files changed, 118 insertions(+), 290 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2024-11-03 02:32:56 Re: pgsql: Move I/O before the index_update_stats() buffer lock region.
Previous Message Bruce Momjian 2024-11-01 20:38:44 pgsql: doc: clarify text around MVCC example query