You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by "C. Michael Pilato" <cm...@collab.net> on 2013/06/20 14:31:27 UTC

Re: svn commit: r1494913 - /subversion/trunk/subversion/libsvn_fs_fs/tree.c

On 06/20/2013 06:31 AM, stefan2@apache.org wrote:
> Author: stefan2
> Date: Thu Jun 20 10:31:53 2013
> New Revision: 1494913
> 
> URL: http://svn.apache.org/r1494913
> Log:
> Fix a severe performance regression in 'svn log':  The new log code
> will run svn_client__repos_location_segments even if we log on the
> root folder.  That triggered a long-standing performance issue in
> fs_node_origin_rev.  This patch fixes the latter.
> 
> * subversion/libsvn_fs_fs/tree.c
>   (fs_node_origin_rev): special case root node IDs instead of walking

Make the same change in the BDB backend?  base_node_origin_rev() looks to be
the spot.

-- 
C. Michael Pilato <cm...@collab.net>
CollabNet   <>   www.collab.net   <>   Enterprise Cloud Development


Re: svn commit: r1494913 - /subversion/trunk/subversion/libsvn_fs_fs/tree.c

Posted by Stefan Fuhrmann <st...@wandisco.com>.
On Thu, Jun 20, 2013 at 2:31 PM, C. Michael Pilato <cm...@collab.net>wrote:

> On 06/20/2013 06:31 AM, stefan2@apache.org wrote:
> > Author: stefan2
> > Date: Thu Jun 20 10:31:53 2013
> > New Revision: 1494913
> >
> > URL: http://svn.apache.org/r1494913
> > Log:
> > Fix a severe performance regression in 'svn log':  The new log code
> > will run svn_client__repos_location_segments even if we log on the
> > root folder.  That triggered a long-standing performance issue in
> > fs_node_origin_rev.  This patch fixes the latter.
> >
> > * subversion/libsvn_fs_fs/tree.c
> >   (fs_node_origin_rev): special case root node IDs instead of walking
>
> Make the same change in the BDB backend?  base_node_origin_rev() looks to
> be
> the spot.
>

r1494967 should do the trick.

-- Stefan^2.