Re: fd.c doesn't remove files on a crash-restart

From: Peter Geoghegan <pg(at)heroku(dot)com>
To: "Joshua D(dot) Drake" <jd(at)commandprompt(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: fd.c doesn't remove files on a crash-restart
Date: 2016-03-16 19:19:39
Message-ID: CAM3SWZREPVKc80Dog+R2sxW=wt+LfCjTyCBPStWo4yA3+zigiA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Mar 16, 2016 at 10:53 AM, Joshua D. Drake <jd(at)commandprompt(dot)com> wrote:
> fd.c[1] will remove files from pgsql_tmp on a restart but not a
> crash-restart per this comment:
>
> /*
> * NOTE: we could, but don't, call this during a post-backend-crash restart
> * cycle. The argument for not doing it is that someone might want to
> examine
> * the temp files for debugging purposes. This does however mean that
> * OpenTemporaryFile had better allow for collision with an existing temp
> * file name.
> */
>
> I understand that this is designed this way. I think it is a bad idea

FWIW, I've seen this get out of hand several times myself.

--
Peter Geoghegan

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2016-03-16 19:25:37 Re: Performance degradation in commit ac1d794
Previous Message Tom Lane 2016-03-16 19:16:30 Re: fd.c doesn't remove files on a crash-restart