Opened 13 years ago
Last modified 13 years ago
#368 new enhancement
Reduce number svn blame calls
Reported by: | Peter Johansson | Owned by: | Peter Johansson |
---|---|---|---|
Priority: | major | Milestone: | svndigest 0.x+ |
Component: | statistics | Version: | trunk |
Keywords: | Cc: |
Description (last modified by )
related to #358
svn blame calls are very expensive; try with svn-client.
It depends, of course, on the connection to the server how large fraction of the run time that we can blame on SVNblame, but I think we can reduce the total run time significantly by minimizing number of calls to SVNblame.
There are two places to reduce number of svn blame:
- obsolete The first place is in the output (as described in ticket:358). If we wanna keep the output as now, we should at least move the svn blame call so it is only done once for each File and then pass around the SVNblame object to the several outputs.
- In the parsing, one svn blame is called for every rev the File has been modified. However, this happens twice since it happens for both SVNblame and SVNadd. If we could re-structure the code, so only one svn blame call is needed and that information can be passed to both classes.
Change History (3)
comment:1 Changed 13 years ago by
Description: | modified (diff) |
---|
comment:2 Changed 13 years ago by
Milestone: | svndigest 0.7 → svndigest 0.x+ |
---|
The loop over the different outputs for one page occur in Node, and therefore it would be a bit complicated to implement 1). As ticket:358 include the same reduction of SVNblame call and it seems to be easier to implement, I'll go for ticket:358 instead.
2) might still be an idea to consider, so I leave this ticket open.
comment:3 Changed 13 years ago by
Description: | modified (diff) |
---|
spelling