[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Issue #1429 et al, WAS: RE: Paul, info for releasing 0.28

From: Sander Striker <striker_at_apache.org>
Date: 2003-08-24 00:36:04 CEST

> From: kfogel@collab.net [mailto:kfogel@collab.net]
> Sent: Friday, August 22, 2003 8:44 PM

> Hi, Paul, welcome back. As you know, there's one issue left in 0.28:
> issue #1429 ("slow checkouts"). Sander Striker's been working on it,
> and has learned that it's not a matter of one single factor that we
> can just *zap* to get faster checkouts. It's about making several
> improvements.

Indeed it is. libsvn_wc and libsvn_ra_dav are on the list. For the
rest it is about overall little inefficiencies accumulating.
 
> He's going to file a separate new issue for each improvement, and then
> use Saturday to work on one of them (probably some libsvn_wc
> efficiency improvements).

My profile* just showed some data that made me freak, since it tells
me it has a function, apr_allocator_set_mutex, that's being called
about a half a million times, but there is no reference in the source
anywhere(!). It appears that it happens when I let gprof attribute
costs to the exported api alone (the -a switch).

I'm going to call it a night, sleep on it and give it another go in
the morning.

From what I've seen this week, the xml related code was pretty prominently
present when added up. As a data point: xml_escape_attr alone was taking
almost 2% of the total time.
Anyway, moving from xml to something else isn't something I'll be attacking
in a measly sunday morning...

Another thing is the md5 checksums. We might want to think about a
little less CPU intensive algorithm, since this one is up there aswell.
MD5Transform (which is internal to apr, called by apr_md5_update) is taking
~15% of total checkout time over ra_local. It's at the very top of the
flat profile.

> So watch issue #1429, and the commit logs.
>
> I guess if Sander has any particular instructions about which revision
> to use (or not use) for the release branch, he'll post them here.

Just use rev 6841. Any changes I may be committing this weekend
would have too little world exposure to include them in the release.

Sander

*) checkout over ra_local of the mozilla tree. The subversion tree
   is too small to see differences between times for a lot of calls,
   hence the large tree.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Aug 24 00:36:57 2003

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.