Cleanup VirtualXact at end of Hot Standby.

simonat2ndQuadrant · simonat2ndQuadrant · commit f1e57a4ec9c0 · 2012-11-29T21:59:11.000Z
diff --git a/src/backend/storage/ipc/standby.c b/src/backend/storage/ipc/standby.c
@@ -105,6 +105,9 @@ ShutdownRecoveryTransactionEnvironment(void)
 
 	/* Release all locks the tracked transactions were holding */
 	StandbyReleaseAllLocks();
+
+	/* Cleanup our VirtualTransaction */
+	VirtualXactLockTableCleanup();
 }
 
 
diff --git a/src/backend/storage/lmgr/lock.c b/src/backend/storage/lmgr/lock.c
@@ -210,7 +210,6 @@ static bool FastPathUnGrantRelationLock(Oid relid, LOCKMODE lockmode);
 static bool FastPathTransferRelationLocks(LockMethod lockMethodTable,
 							  const LOCKTAG *locktag, uint32 hashcode);
 static PROCLOCK *FastPathGetRelationLockEntry(LOCALLOCK *locallock);
-static void VirtualXactLockTableCleanup(void);
 
 /*
  * To make the fast-path lock mechanism work, we must have some way of
@@ -3791,7 +3790,7 @@ VirtualXactLockTableInsert(VirtualTransactionId vxid)
  *		Check whether a VXID lock has been materialized; if so, release it,
  *		unblocking waiters.
  */
-static void
+void
 VirtualXactLockTableCleanup()
 {
 	bool		fastpath;
diff --git a/src/include/storage/lock.h b/src/include/storage/lock.h
@@ -544,6 +544,7 @@ extern void DumpAllLocks(void);
 
 /* Lock a VXID (used to wait for a transaction to finish) */
 extern void VirtualXactLockTableInsert(VirtualTransactionId vxid);
+extern void VirtualXactLockTableCleanup(void);
 extern bool VirtualXactLock(VirtualTransactionId vxid, bool wait);
 
 #endif   /* LOCK_H */

Original file line number	Diff line number	Diff line change
`@@ -105,6 +105,9 @@ ShutdownRecoveryTransactionEnvironment(void)`
`105`	`105`
`106`	`106`	`/* Release all locks the tracked transactions were holding */`
`107`	`107`	`StandbyReleaseAllLocks();`
	`108`	`+`
	`109`	`+ /* Cleanup our VirtualTransaction */`
	`110`	`+ VirtualXactLockTableCleanup();`
`108`	`111`	`}`
`109`	`112`
`110`	`113`