Oops, fix recoveryStopsBefore functions for regular commits.

hlinnaka · hlinnaka · commit 5b7f1f521079 · 2014-07-29T17:26:36.000+03:00
Pointed out by Tom Lane. Backpatch to 9.4, the code was structured
differently in earlier branches and didn't have this mistake.
diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c
@@ -5489,7 +5489,7 @@ recoveryStopsBefore(XLogRecord *record)
 		isCommit = true;
 		recordXid = record->xl_xid;
 	}
-	if (record_info == XLOG_XACT_COMMIT_PREPARED)
+	else if (record_info == XLOG_XACT_COMMIT_PREPARED)
 	{
 		isCommit = true;
 		recordXid = ((xl_xact_commit_prepared *) XLogRecGetData(record))->xid;

Original file line number	Diff line number	Diff line change
`@@ -5489,7 +5489,7 @@ recoveryStopsBefore(XLogRecord *record)`
`5489`	`5489`	`isCommit = true;`
`5490`	`5490`	`recordXid = record->xl_xid;`
`5491`	`5491`	`}`
`5492`		`- if (record_info == XLOG_XACT_COMMIT_PREPARED)`
	`5492`	`+ else if (record_info == XLOG_XACT_COMMIT_PREPARED)`
`5493`	`5493`	`{`
`5494`	`5494`	`isCommit = true;`
`5495`	`5495`	`recordXid = ((xl_xact_commit_prepared *) XLogRecGetData(record))->xid;`