Remove dead isXmax argument to Snapshot functions.

src/backend/cdb/cdbdistributedsnapshot.c

@@ -27,8 +27,7 @@
 DistributedSnapshotCommitted 
 DistributedSnapshotWithLocalMapping_CommittedTest(
 	DistributedSnapshotWithLocalMapping		*dslm,
-	TransactionId 							localXid,
-	bool									isXmax)
+	TransactionId 							localXid)
 {
 	DistributedSnapshotHeader *header = &dslm->header;
 	DistributedSnapshotMapEntry *inProgressEntryArray = dslm->inProgressEntryArray;

src/backend/utils/time/tqual.c

@@ -117,9 +117,9 @@ TransactionId RecentGlobalXmin = InvalidTransactionId;
 
 
 /* local functions */
-static bool XidInMVCCSnapshot(TransactionId xid, Snapshot snapshot, bool isXmax,
+static bool XidInMVCCSnapshot(TransactionId xid, Snapshot snapshot,
 			  bool distributedSnapshotIgnore, bool *setDistributedSnapshotIgnore);
-static bool XidInMVCCSnapshot_Local(TransactionId xid, Snapshot snapshot, bool isXmax);
+static bool XidInMVCCSnapshot_Local(TransactionId xid, Snapshot snapshot);
 
 /*
  * Set the buffer dirty after setting t_infomask
@@ -1112,7 +1112,6 @@ HeapTupleSatisfiesMVCC(Relation relation, HeapTupleHeader tuple, Snapshot snapsh
 	 */
 	inSnapshot =
 		XidInMVCCSnapshot(HeapTupleHeaderGetXmin(tuple), snapshot,
-						  /* isXmax */ false,
 						  ((tuple->t_infomask2 & HEAP_XMIN_DISTRIBUTED_SNAPSHOT_IGNORE) != 0),
 						  &setDistributedSnapshotIgnore);
 	if (setDistributedSnapshotIgnore)
@@ -1168,7 +1167,6 @@ HeapTupleSatisfiesMVCC(Relation relation, HeapTupleHeader tuple, Snapshot snapsh
 	 */
 	inSnapshot =
 			XidInMVCCSnapshot(HeapTupleHeaderGetXmax(tuple), snapshot,
-							  /* isXmax */ true,
 							  ((tuple->t_infomask2 & HEAP_XMAX_DISTRIBUTED_SNAPSHOT_IGNORE) != 0),
 							  &setDistributedSnapshotIgnore);
 	if (setDistributedSnapshotIgnore)
@@ -1531,7 +1529,7 @@ FreeXactSnapshot(void)
  *      and local snapshots?
  */
 static bool
-XidInMVCCSnapshot(TransactionId xid, Snapshot snapshot, bool isXmax,
+XidInMVCCSnapshot(TransactionId xid, Snapshot snapshot,
 				  bool distributedSnapshotIgnore, bool *setDistributedSnapshotIgnore)
 {
 	Assert (setDistributedSnapshotIgnore != NULL);
@@ -1555,8 +1553,7 @@ XidInMVCCSnapshot(TransactionId xid, Snapshot snapshot, bool isXmax,
 		distributedSnapshotCommitted =
 			DistributedSnapshotWithLocalMapping_CommittedTest(
 				&snapshot->distribSnapshotWithLocalMapping,
-				xid,
-				isXmax);
+				xid);
 
 		switch (distributedSnapshotCommitted)
 		{
@@ -1581,7 +1578,7 @@ XidInMVCCSnapshot(TransactionId xid, Snapshot snapshot, bool isXmax,
 		}
 	}
 
-	return XidInMVCCSnapshot_Local(xid, snapshot, isXmax);
+	return XidInMVCCSnapshot_Local(xid, snapshot);
 }
 
 /*
@@ -1594,7 +1591,7 @@ XidInMVCCSnapshot(TransactionId xid, Snapshot snapshot, bool isXmax,
  * apply this for known-committed XIDs.
  */
 static bool
-XidInMVCCSnapshot_Local(TransactionId xid, Snapshot snapshot, bool isXmax)
+XidInMVCCSnapshot_Local(TransactionId xid, Snapshot snapshot)
 {
 	uint32		i;
 

src/include/cdb/cdbdistributedsnapshot.h

@@ -115,8 +115,7 @@ typedef enum
 
 extern DistributedSnapshotCommitted DistributedSnapshotWithLocalMapping_CommittedTest(
 	DistributedSnapshotWithLocalMapping		*dslm,
-	TransactionId 							localXid,
-	bool									isXmax);
+	TransactionId 							localXid);
 
 extern void DistributedSnapshot_Reset(
 	DistributedSnapshot *distributedSnapshot);