On Tue, 2007-01-02 at 18:20 -0500, Tom Lane wrote:
> Bill Moran <wmoran(at)collaborativefusion(dot)com> writes:
> > In response to Alvaro Herrera <alvherre(at)commandprompt(dot)com>:
> >> Please change things to save the stat() syscall when the feature is not
> >> in use.
> 
> > Do you have a suggestion on how to do that and still have the PG_TRACE1()
> > work?  That was specifically requested by Simon Riggs.
> 
> Well, we are NOT paying a stat() call on every single file close,
> whether Simon wants it or not.  

Simon doesn't/wouldn't want the stat() call on each file close.

If you put the PG_TRACE macro outside of the if test, yet prior to the
file close, you can pass the filename through like this

	PG_TRACE1(temp__file__cleanup, vfdP->fileName);

That way DTrace can make its own call to find out filesize, if it would
like to... and we don't need to stat() before each temp file close.
That's much more flexible and useful, as well as better performance.

-- 
  Simon Riggs             
  EnterpriseDB   http://www.enterprisedb.com

In response to

• Re: Patch to log usage of temporary files at 2007-01-02 23:20:37 from Tom Lane

Responses

• Re: [HACKERS] Patch to log usage of temporary files at 2007-01-03 21:20:53 from Bill Moran

pgsql-hackers by date

Next:	From: Jim Nasby	Date: 2007-01-03 15:40:38
	Subject: Re: TODO: Add a GUC to control whether BEGIN inside
Previous:	From: Jim Nasby	Date: 2007-01-03 15:33:35
	Subject: Re: effective_cache_size vs units

pgsql-patches by date

Next:	From: Bill Moran	Date: 2007-01-03 21:20:53
	Subject: Re: [HACKERS] Patch to log usage of temporary files
Previous:	From: Zeugswetter Andreas ADI SD	Date: 2007-01-03 11:18:42
	Subject: Re: Load distributed checkpoint