Fix position of WalSndWakeupRequest call.

robertmhaas · robertmhaas · commit 3bb592bb20d0 · 2012-07-02T14:44:10.000-04:00
This avoids discriminating against wal_sync_method = open_sync or
open_datasync.

Fujii Masao, reviewed by Andres Freund
diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c
@@ -1867,10 +1867,11 @@ XLogWrite(XLogwrtRqst WriteRqst, bool flexible, bool xlog_switch)
 			}
 
 			issue_xlog_fsync(openLogFile, openLogSegNo);
-
-			/* signal that we need to wakeup walsenders later */
-			WalSndWakeupRequest();
 		}
+
+		/* signal that we need to wakeup walsenders later */
+		WalSndWakeupRequest();
+
 		LogwrtResult.Flush = LogwrtResult.Write;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -1867,10 +1867,11 @@ XLogWrite(XLogwrtRqst WriteRqst, bool flexible, bool xlog_switch)`
`1867`	`1867`	`}`
`1868`	`1868`
`1869`	`1869`	`issue_xlog_fsync(openLogFile, openLogSegNo);`
`1870`		`-`
`1871`		`- /* signal that we need to wakeup walsenders later */`
`1872`		`- WalSndWakeupRequest();`
`1873`	`1870`	`}`
	`1871`	`+`
	`1872`	`+ /* signal that we need to wakeup walsenders later */`
	`1873`	`+ WalSndWakeupRequest();`
	`1874`	`+`
`1874`	`1875`	`LogwrtResult.Flush = LogwrtResult.Write;`
`1875`	`1876`	`}`
`1876`	`1877`