Repair longstanding bug in slru/clog logic: it is possible for two backends
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 21 Jan 2006 04:38:21 +0000 (04:38 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 21 Jan 2006 04:38:21 +0000 (04:38 +0000)
commitc89a0dd3bb2c1ab5d375335fee1af523e73f79cb
tree76a082fd64adcb063ad7cd76c4d7a66325c9a3c4
parent4789e9880148660c7126aef4fbaf5563be6ff167
Repair longstanding bug in slru/clog logic: it is possible for two backends
to try to create a log segment file concurrently, but the code erroneously
specified O_EXCL to open(), resulting in a needless failure.  Before 7.4,
it was even a PANIC condition :-(.  Correct code is actually simpler than
what we had, because we can just say O_CREAT to start with and not need a
second open() call.  I believe this accounts for several recent reports of
hard-to-reproduce "could not create file ...: File exists" errors in both
pg_clog and pg_subtrans.
src/backend/access/transam/slru.c