Re: Reviewing temp_tablespaces GUC patch - Mailing list pgsql-hackers

From Jaime Casanova
Subject Re: Reviewing temp_tablespaces GUC patch
Date
Msg-id c2d9e70e0705291706v1ee431demd394877297759026@mail.gmail.com
Whole thread Raw
In response to Re: Reviewing temp_tablespaces GUC patch  (Robert Treat <xzilla@users.sourceforge.net>)
Responses Re: Reviewing temp_tablespaces GUC patch  (Robert Treat <xzilla@users.sourceforge.net>)
List pgsql-hackers
On 5/27/07, Robert Treat <xzilla@users.sourceforge.net> wrote:
> On Friday 25 May 2007 12:39, Jaime Casanova wrote:
> > On 5/25/07, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> > > Bernd Helmle <mailings@oopsware.de> writes:
> > > > --On Freitag, Mai 25, 2007 10:49:29 +0000 Jaime Casanova
> > > >
> > > > <systemguards@gmail.com> wrote:
> > > >> No, because the RemovePgTempFiles() call in PostmasterMain() will
> > > >> remove all tmp files at startup.
> > >
> > > I believe we do not call RemovePgTempFiles during a crash recovery
> > > cycle; this is intentional on the theory that the temp files might
> > > contain useful debugging clues.
> >
> > ah, i forgot that
> >
> > >  So there is a potential problem there.
> > > Not sure how important it really is though --- neither crashes nor
> > > tablespace drops ought to be so common that we need a really nice
> > > solution.
> >
> > the only semi-sane solution i can think of, is to have a superuser
> > only function that acts as a wrapper for RemovePgTempFiles(), but
> > still exists a chance for shoot yourself on the foot...
>
> If there was a way for DBA's to know they could safely delete the left-over
> files (maybe the files timestamp is older than postmaster start; though not
> sure how you measure that), then I think this would be enough to give them a
> way out.  Of course maybe that level of smarts could be put into drop
> tablespace itself?
>

i don't think silently delete the files is a good idea, specially if
the files are left there intencionally...

but what, exactly, we want to do? delete the files or maybe sending an
HINT just after the error so we can inform the DBA about the temp
files and let him decide.

comments?

BTW, postmaster startup is in PgStartTime, right?

-- 
regards,
Jaime Casanova

"Programming today is a race between software engineers striving to
build bigger and better idiot-proof programs and the universe trying
to produce bigger and better idiots.
So far, the universe is winning."                                      Richard Cook


pgsql-hackers by date:

Previous
From: "Andrew Hammond"
Date:
Subject: Re: interval / interval -> double operator
Next
From: Alvaro Herrera
Date:
Subject: Re: interval / interval -> double operator