Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 0a971e2

Browse files
committed
Disable full_page_writes, because turning it off risks causing crash-recovery
failures even when the hardware and OS did nothing wrong. Per recent analysis of a problem report from Alex Bahdushka. For the moment I've just diked out the test of the parameter, rather than removing the GUC infrastructure and documentation, in case we conclude that there's something salvageable there. There seems no chance of it being resurrected in the 8.1 branch though.
1 parent 288551f commit 0a971e2

File tree

1 file changed

+2
-3
lines changed
  • src/backend/access/transam

1 file changed

+2
-3
lines changed

src/backend/access/transam/xlog.c

+2-3
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
* Portions Copyright (c) 1996-2006, PostgreSQL Global Development Group
88
* Portions Copyright (c) 1994, Regents of the University of California
99
*
10-
* $PostgreSQL: pgsql/src/backend/access/transam/xlog.c,v 1.228 2006/03/24 04:32:13 tgl Exp $
10+
* $PostgreSQL: pgsql/src/backend/access/transam/xlog.c,v 1.229 2006/03/28 22:01:16 tgl Exp $
1111
*
1212
*-------------------------------------------------------------------------
1313
*/
@@ -980,8 +980,7 @@ XLogCheckBuffer(XLogRecData *rdata,
980980
*/
981981
*lsn = page->pd_lsn;
982982

983-
if (fullPageWrites &&
984-
XLByteLE(page->pd_lsn, RedoRecPtr))
983+
if (XLByteLE(page->pd_lsn, RedoRecPtr))
985984
{
986985
/*
987986
* The page needs to be backed up, so set up *bkpb

0 commit comments

Comments
 (0)