Fix a typo, and remove unnecessary pgstat_report_wait_end().

robertmhaas · robertmhaas · commit 481c76abf4f5 · 2016-03-11T07:34:00.000-05:00
Per Amit Kapila.
diff --git a/src/backend/access/transam/xact.c b/src/backend/access/transam/xact.c
@@ -4655,9 +4655,6 @@ AbortSubTransaction(void)
 	 */
 	XactReadOnly = s->prevXactReadOnly;
 
-	/* Report wait end here, when there is no further possibility of wait */
-	pgstat_report_wait_end();
-
 	RESUME_INTERRUPTS();
 }
 
diff --git a/src/backend/storage/lmgr/lwlock.c b/src/backend/storage/lmgr/lwlock.c
@@ -760,9 +760,8 @@ GetLWLockIdentifier(uint8 classId, uint16 eventId)
 
 	/*
 	 * It is quite possible that user has registered tranche in one of the
-	 * backends (e.g. by allocation lwlocks in dynamic shared memory) but not
-	 * all of them, so we can't assume the tranche is registered here.
-	 * extension for such cases.
+	 * backends (e.g. by allocating lwlocks in dynamic shared memory) but
+	 * not all of them, so we can't assume the tranche is registered here.
 	 */
 	if (eventId >= LWLockTranchesAllocated ||
 		LWLockTrancheArray[eventId]->name == NULL)

Original file line number	Diff line number	Diff line change
`@@ -4655,9 +4655,6 @@ AbortSubTransaction(void)`
`4655`	`4655`	`*/`
`4656`	`4656`	`XactReadOnly = s->prevXactReadOnly;`
`4657`	`4657`
`4658`		`- /* Report wait end here, when there is no further possibility of wait */`
`4659`		`- pgstat_report_wait_end();`
`4660`		`-`
`4661`	`4658`	`RESUME_INTERRUPTS();`
`4662`	`4659`	`}`
`4663`	`4660`