Re: pgsql: Further thoughts about temp_file_limit patch. - Mailing list pgsql-committers

From Cédric Villemain
Subject Re: pgsql: Further thoughts about temp_file_limit patch.
Date
Msg-id CAF6yO=2nwzvk2OtQr8ubx6jjXG1PQPG_fiX-XLtwBGXeMZB_SA@mail.gmail.com
Whole thread Raw
In response to pgsql: Further thoughts about temp_file_limit patch.  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: pgsql: Further thoughts about temp_file_limit patch.
List pgsql-committers
2011/7/17 Tom Lane <tgl@sss.pgh.pa.us>:
> Further thoughts about temp_file_limit patch.
>
> Move FileClose's decrement of temporary_files_size up, so that it will be
> executed even if elog() throws an error.  This is reasonable since if the
> unlink() fails, the fact the file is still there is not our fault, and we
> are going to forget about it anyhow.  So we won't count it against
> temp_file_limit anymore.
>
> Update fileSize and temporary_files_size correctly in FileTruncate.
> We probably don't have any places that truncate temp files, but fd.c
> surely should not assume that.
>
> Branch
> ------
> master
>
> Details
> -------
> http://git.postgresql.org/pg/commitdiff/9473bb96d0eb7ed73f1bf5269613e6266f64ad85
>
> Modified Files
> --------------
> src/backend/storage/file/fd.c |   17 +++++++++++++----
> 1 files changed, 13 insertions(+), 4 deletions(-)
>
>
> --
> Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-committers
>


How does it sound to remove the stat() call used when log_temp_file is
set in favor of fileSize that we now have ?
(I didn't check the code rigth now but the 2 patches do not look to
touch that part of the code)

--
Cédric Villemain               2ndQuadrant
http://2ndQuadrant.fr/     PostgreSQL : Expertise, Formation et Support

pgsql-committers by date:

Previous
From: Robert Haas
Date:
Subject: pgsql: Remove superfluous variable.
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Further thoughts about temp_file_limit patch.