]> granicus.if.org Git - postgresql/commit
Improve error handling in RemovePgTempFiles().
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 4 Dec 2017 22:59:35 +0000 (17:59 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 4 Dec 2017 22:59:35 +0000 (17:59 -0500)
commit561885db05d3296082ce8750805b8ec322cf9aa1
tree21c791d9a03a2b43d265d9ebfd9502aeda9ba886
parent2069e6faa0f72eba968714b2260cd65436d0ef3a
Improve error handling in RemovePgTempFiles().

Modify this function and its subsidiaries so that syscall failures are
reported via ereport(LOG), rather than silently ignored as before.
We don't want to throw a hard ERROR, as that would prevent database
startup, and getting rid of leftover temporary files is not important
enough for that.  On the other hand, not reporting trouble at all
seems like an odd choice not in line with current project norms,
especially since any failure here is quite unexpected.

On the same reasoning, adjust these functions' AllocateDir/ReadDir calls
so that failure to scan a directory results in LOG not ERROR.  I also
removed the previous practice of silently ignoring ENOENT failures during
directory opens --- there are some corner cases where that could happen
given a previous database crash, but that seems like a bad excuse for
ignoring a condition that isn't expected in most cases.  A LOG message
during postmaster start seems OK in such situations, and better than
no output at all.

In passing, make RemovePgTempRelationFiles' test for "is the file name
all digits" look more like the way it's done elsewhere.

Discussion: https://postgr.es/m/19907.1512402254@sss.pgh.pa.us
src/backend/storage/file/fd.c