apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Jagielski <...@devsys.jaguNET.com>
Subject Re: svn commit: r512557 - in /apr/apr-util/trunk: CHANGES dbm/sdbm/sdbm.c
Date Wed, 28 Feb 2007 16:53:45 GMT
Joe Orton wrote:
> 
> On Wed, Feb 28, 2007 at 08:29:10AM -0800, Justin Erenkrantz wrote:
> > The performance implications of not doing buffered reads just *kills*
> > our server - so we need to do something and adding buffering lessens
> > the load quite dramatically.  In our situation, we couldn't care less
> > about modifying the file - we only care to optimize the read-only case
> > - and I believe that code is just fine and stable.  Though I guess I'd
> > prefer we fix the problems with buffering if they do exist.
> > 
> > Would you be concerned if we added APR_BUFFERED to APR_DBM_DBMODE_RO
> > for sdbm?  -- justin
> 
> No, that sounds fine, or just moving your change up inside the !(flags & 
> APR_WRITE) condition so that all apr_sdbm_* users benefit equally.
> 

+1

-- 
===========================================================================
   Jim Jagielski   [|]   jim@jaguNET.com   [|]   http://www.jaguNET.com/
	    "If you can dodge a wrench, you can dodge a ball."

Mime
View raw message