pgsql: Improve underdocumented btree_xlog_delete_get_latestRemovedXid() - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Improve underdocumented btree_xlog_delete_get_latestRemovedXid()
Date
Msg-id E1SxNkb-0004Fm-32@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Improve underdocumented btree_xlog_delete_get_latestRemovedXid() code.

As noted by Noah Misch, btree_xlog_delete_get_latestRemovedXid is
critically dependent on the assumption that it's examining a consistent
state of the database.  This was undocumented though, so the
seemingly-unrelated check for no active HS sessions might be thought to be
merely an optional optimization.  Improve comments, and add an explicit
check of reachedConsistency just to be sure.

This function returns InvalidTransactionId (thereby killing all HS
transactions) in several cases that are not nearly unlikely enough for my
taste.  This commit doesn't attempt to fix those deficiencies, just
document them.

Back-patch to 9.2, not from any real functional need but just to keep the
branches more closely synced to simplify possible future back-patching.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/f786e91a75b2f64527dcf321e754b6448fcad7fe

Modified Files
--------------
src/backend/access/nbtree/nbtxlog.c |   41 ++++++++++++++++++++++++++--------
1 files changed, 31 insertions(+), 10 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: In SPGiST replay, do conflict resolution before modifying the pa
Next
From: Bruce Momjian
Date:
Subject: pgsql: Add link to synchronous_commit variables in high availability do