Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#358 closed task (fixed)

Avoid replicating blame output

Reported by: Peter Johansson Owned by: Peter Johansson
Priority: critical Milestone: svndigest 0.7
Component: output Version: trunk
Keywords: Cc:

Description (last modified by Peter Johansson)

related to ticket:334, ticket:296, ticket:330, ticket:368

Feature competes with ticket:269

In every File output contains a blame output and the thing is that the blame output is the same no matter what of author, LineType, and StatsType. Therefore it would be possible to, rather than write this multiple times, only write the blame output once and link to the output from each File output.

For each File there are output for 3 StatsType times 4 LineType times number of authors (plus all). So for a small project like this each file has output 36 times so we could save a lot in terms of output disc space. But the save is also in runtime because writing all that output takes quite some time. And it is not only that it takes time to write to disc, it is also that every time a call to SVNblame is needed. Therefore, if we'd like to keep the output like it is, we could at least avoid calling SVNblame multiple times for the replicates.

Another this is that one perhaps should only write the blame output when it has been updated but that feature belong in ticket:334, I guess.

Change History (8)

comment:1 Changed 12 years ago by Peter Johansson

Description: modified (diff)

comment:2 Changed 12 years ago by Peter Johansson

Description: modified (diff)

adding relationship to #296 and #330 changing relationship of #269 to competing

comment:3 Changed 12 years ago by Peter Johansson

Description: modified (diff)

comment:4 Changed 12 years ago by Peter Johansson

Description: modified (diff)

ticket:368 was marked as related

comment:5 Changed 12 years ago by Peter Johansson

Milestone: svndigest 0.x+svndigest 0.7
Status: newassigned

comment:6 Changed 12 years ago by Peter Johansson

Resolution: fixed
Status: assignedclosed

(In [750]) fixes #358

comment:7 Changed 12 years ago by Peter Johansson

Description: modified (diff)

removed competition with #161

comment:8 Changed 12 years ago by Jari Häkkinen

(In [755]) Addresses #358. Added by strerror() required include statements (mac, SuSE)

Note: See TracTickets for help on using tickets.