On Fri, 2009-08-07 at 19:29 +0000, Tom Lane wrote: > Log Message: > ----------- > rm_cleanup functions need to be allowed to write WAL entries. This oversight > appears to explain the recent reports of "PANIC: cannot make new WAL entries > during recovery". > > Modified Files: > -------------- > pgsql/src/backend/access/transam: > xlog.c (r1.345 -> r1.346) > > (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/access/transam/xlog.c?r1=1.345&r2=1.346)
I was just working on a patch after the comments yesterday, hadn't noticed you'd committed. The first chunk is exactly as I was going to suggest. Make an explicit state change in the startup process. Resetting it back seems fragile, since in crash recovery we call it again almost immediately during CreateCheckPoint(). That only works if LocalSetXLogInsertAllowed() has no side effects. I understand Heikki's wish to have safeguards in place, so we should document that LocalSetXLogInsertAllowed() can be executed twice without problem. -- Simon Riggs www.2ndQuadrant.com -- Sent via pgsql-committers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-committers
