On Tue, Dec 5, 2017 at 4:44 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Anyway, I'm inclined to reverse that choice and emit LOG messages
> reporting failure of any of the lstat, rmdir, or unlink calls in
> RemovePgTempFilesInDir. In the worst case, say that there are a
> bunch of leftover temp files in a directory that someone has made
> unwritable, this might cause a fair amount of whining in the postmaster
> log --- but that's a situation that needs to be fixed anyway, so
> I cannot see how not printing anything is a good idea.
Belatedly, +1. The error hiding seemed a bit odd considering that we
were prepared to log "unexpected file found ...". I probably should
have questioned that instead of extending it monkey-see-monkey-do.
--
Thomas Munro
http://www.enterprisedb.com