Add missing Assert to make TransactionIdDidAbort more consistent with

tglsfdc · tglsfdc · commit 7531d2fd85c0 · 2004-08-28T21:58:59.000Z
TransactionIdDidCommit.
diff --git a/src/backend/access/transam/transam.c b/src/backend/access/transam/transam.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/access/transam/transam.c,v 1.58 2004/08/22 02:41:57 tgl Exp $
+ *	  $PostgreSQL: pgsql/src/backend/access/transam/transam.c,v 1.59 2004/08/28 21:58:59 tgl Exp $
  *
  * NOTES
  *	  This file contains the high level access-method interface to the
@@ -258,6 +258,7 @@ TransactionIdDidAbort(TransactionId transactionId)
 		if (TransactionIdPrecedes(transactionId, RecentXmin))
 			return true;
 		parentXid = SubTransGetParent(transactionId);
+		Assert(TransactionIdIsValid(parentXid));
 		return TransactionIdDidAbort(parentXid);
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`*`
`9`	`9`	`*`
`10`	`10`	`* IDENTIFICATION`
`11`		`- * $PostgreSQL: pgsql/src/backend/access/transam/transam.c,v 1.58 2004/08/22 02:41:57 tgl Exp $`
	`11`	`+ * $PostgreSQL: pgsql/src/backend/access/transam/transam.c,v 1.59 2004/08/28 21:58:59 tgl Exp $`
`12`	`12`	`*`
`13`	`13`	`* NOTES`
`14`	`14`	`* This file contains the high level access-method interface to the`
`@@ -258,6 +258,7 @@ TransactionIdDidAbort(TransactionId transactionId)`
`258`	`258`	`if (TransactionIdPrecedes(transactionId, RecentXmin))`
`259`	`259`	`return true;`
`260`	`260`	`parentXid = SubTransGetParent(transactionId);`
	`261`	`+ Assert(TransactionIdIsValid(parentXid));`
`261`	`262`	`return TransactionIdDidAbort(parentXid);`
`262`	`263`	`}`
`263`	`264`