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 2011/03/18 04:20:46 UTC

Re: svn commit: r1082806 - in /subversion/trunk: notes/wc_node_walkers.txt subversion/include/private/svn_wc_private.h subversion/libsvn_client/ra.c subversion/libsvn_wc/node.c

On 03/17/2011 10:52 PM, cmpilato@apache.org wrote:
> Author: cmpilato
> Date: Fri Mar 18 02:52:57 2011
> New Revision: 1082806
> 
> URL: http://svn.apache.org/viewvc?rev=1082806&view=rev
> Log:
> Eliminate another node walk, this time in the invalidate_wc_prop() RA
> callback function.

In a sparse working copy of the Subversion codebase (/trunk,
/branches/1.6.x, /branches/1.5.x, and /site), here's how the cost of a no-op
svn switch (to the same URL and revision) -- which makes use of this
invalidate_wc_prop callback -- compares:

             1.6.x     1.7 (before)   1.7 (after)
        ,-----------------------------------------.
ra_neon |      2s     |     15s     |     2s      |
        |-------------+-------------+-------------|
ra_serf |      2s     |     15s     |     2s      |
        `-----------------------------------------'


-- 
C. Michael Pilato <cm...@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand


Re: svn commit: r1082806 - in /subversion/trunk: notes/wc_node_walkers.txt subversion/include/private/svn_wc_private.h subversion/libsvn_client/ra.c subversion/libsvn_wc/node.c

Posted by Greg Stein <gs...@gmail.com>.
On Thu, Mar 17, 2011 at 23:20, C. Michael Pilato <cm...@collab.net> wrote:
> On 03/17/2011 10:52 PM, cmpilato@apache.org wrote:
>> Author: cmpilato
>> Date: Fri Mar 18 02:52:57 2011
>> New Revision: 1082806
>>
>> URL: http://svn.apache.org/viewvc?rev=1082806&view=rev
>> Log:
>> Eliminate another node walk, this time in the invalidate_wc_prop() RA
>> callback function.
>
> In a sparse working copy of the Subversion codebase (/trunk,
> /branches/1.6.x, /branches/1.5.x, and /site), here's how the cost of a no-op
> svn switch (to the same URL and revision) -- which makes use of this
> invalidate_wc_prop callback -- compares:
>
>             1.6.x     1.7 (before)   1.7 (after)
>        ,-----------------------------------------.
> ra_neon |      2s     |     15s     |     2s      |
>        |-------------+-------------+-------------|
> ra_serf |      2s     |     15s     |     2s      |
>        `-----------------------------------------'

You go, Big Daddy!

Re: svn commit: r1082806 - in /subversion/trunk: notes/wc_node_walkers.txt subversion/include/private/svn_wc_private.h subversion/libsvn_client/ra.c subversion/libsvn_wc/node.c

Posted by Ivan Zhakov <iv...@visualsvn.com>.
On Fri, Mar 18, 2011 at 06:20, C. Michael Pilato <cm...@collab.net> wrote:
> On 03/17/2011 10:52 PM, cmpilato@apache.org wrote:
>> Author: cmpilato
>> Date: Fri Mar 18 02:52:57 2011
>> New Revision: 1082806
>>
>> URL: http://svn.apache.org/viewvc?rev=1082806&view=rev
>> Log:
>> Eliminate another node walk, this time in the invalidate_wc_prop() RA
>> callback function.
>
> In a sparse working copy of the Subversion codebase (/trunk,
> /branches/1.6.x, /branches/1.5.x, and /site), here's how the cost of a no-op
> svn switch (to the same URL and revision) -- which makes use of this
> invalidate_wc_prop callback -- compares:
>
>             1.6.x     1.7 (before)   1.7 (after)
>        ,-----------------------------------------.
> ra_neon |      2s     |     15s     |     2s      |
>        |-------------+-------------+-------------|
> ra_serf |      2s     |     15s     |     2s      |
>        `-----------------------------------------'
>
Wow!


-- 
Ivan Zhakov