You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Blair Zajac <bl...@orcaware.com> on 2008/06/19 21:28:26 UTC

Re: svn commit: r31820 - trunk/subversion/libsvn_fs_base

cmpilato@tigris.org wrote:
> Author: cmpilato
> Date: Thu Jun 19 14:26:29 2008
> New Revision: 31820
> 
> Log:
> Avoid a bunch of unnecessary database queries during the post-commit
> deltification phase in Berkeley DB repositories.
> 
> deltify_mutable(), when recursing, iterates over directory entries,
> calling itself.  The first thing it does is check the provided path's
> node kind and node-revision-ID.  But both of those pieces of
> information are already available in the entries over which the caller
> is iterating!  So, this change just lets the caller pass the
> information it already has on hand into deltify_mutable().  This has
> significant impact in some situations.  On a test repository
> containing a directory of almost 20,000 files, commit which removed 10
> of those files at a time went from spending 13 minutes in delification
> to just a few seconds.

Wow, nice work.

Blair

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org