Re: pgsql: Remove BufFile's isTemp flag. - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: Remove BufFile's isTemp flag.
Date
Msg-id 24110.1510888651@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql: Remove BufFile's isTemp flag.  (Andres Freund <andres@anarazel.de>)
Responses Re: pgsql: Remove BufFile's isTemp flag.
List pgsql-committers
Andres Freund <andres@anarazel.de> writes:
> On 2017-11-16 21:58:14 -0500, Tom Lane wrote:
>> [ squint... ]  That used to have an actual purpose connected to
>> transaction-abort cleanup, IIRC.  It disturbs me that this seems
>> to have been lost.

> I've not found any such use, searching through buffile.c's history. I
> don't quite see how that flag could've been related to abort cleanup
> stuff?  There's been another unused caller of makeBufFile, namely
> BufFileCreate, that has been #ifdef'ed out for ages (perhaps we
> should've removed that with this commit or a long time ago).  Other than
> that there seems to not have been any other caller setting that flag
> differently since you created the file in db3c4c3a2d980dcd.

I'm tired for today, but will take a closer look tomorrow.  I do not
think the mechanism was created without a purpose ...
        regards, tom lane


pgsql-committers by date:

Previous
From: Andres Freund
Date:
Subject: Re: pgsql: Remove BufFile's isTemp flag.
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Remove BufFile's isTemp flag.