pgsql: Don't MultiXactIdIsRunning when in recovery - Mailing list pgsql-committers

From Alvaro Herrera
Subject pgsql: Don't MultiXactIdIsRunning when in recovery
Date
Msg-id E1YuRtd-0004M6-BS@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Don't MultiXactIdIsRunning when in recovery

In 9.1 and earlier, it is possible for index_getnext() to try to examine
a heap buffer for possible HOT-prune when in recovery; this causes a
problem when a multixact is found in a tuple's Xmax, because
GetMultiXactIdMembers refuses to run when in recovery, raising an error:
    ERROR:  cannot GetMultiXactIdMembers() during recovery

This can be solved easily by having MultiXactIdIsRunning always return
false when in recovery, which is reasonable because a HOT standby cannot
acquire further tuple locks nor update/delete tuples.

(Note: it doesn't look like this specific code path has a problem in
9.2, because instead of doing HeapTupleSatisfiesUpdate directly,
heap_hot_search_buffer uses HeapTupleIsSurelyDead instead.  Still, there
may be other paths affected by the same bug, for instance in pgrowlocks,
and the multixact code hasn't changed; so apply the same fix
throughout.)

Apply this fix to 9.0 through 9.2.  In 9.3 the multixact code has been
changed completely and is no longer subject to this problem.

Per report from Marko Tiikkaja,
https://www.postgresql.org/message-id/54EB3283.2080305@joh.to
Analysis by Andres Freund

Branch
------
REL9_1_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/2360eea3be67fb9650067817a4e50fc2f1b8cff7

Modified Files
--------------
src/backend/access/transam/multixact.c |   15 +++++++++++++++
1 file changed, 15 insertions(+)


pgsql-committers by date:

Previous
From: Alvaro Herrera
Date:
Subject: pgsql: Don't MultiXactIdIsRunning when in recovery
Next
From: Tom Lane
Date:
Subject: pgsql: Put back a backwards-compatible version of sampling support func