Re: [Cobertura-devel] Performance Issues - cobertura.ser

[Grzegorz L. <ha...@gm...>]

The problem here may be implemetation of
ProjectData.getOrCreateClassData method, and additionaly
ClassData.touch implementation. Both these methods are invoked each
time "line of code" is executed. Both methods get/put some information
from maps. The result is that with each line of code some operations
that are in most cases many times more expensive are executed.

Grzegorz

On 6/10/05, Nathan Johns <nr...@gm...> wrote:
> Hi,
>=20
> It appears to me, though I haven't looked at the code, that every time
> the cobertura.ser file is touched by a test for instrumentation the
> entire file is read in, updated and then written back to disk.  This
> seems to be a hug performance hit when compared to running the test
> without instrumentation.  (in our case about 6-7 times slower, on a
> build that take 15 minutes without instrumentation this makes it a
> very long build cycle.)
>=20
> Would it perhaps make more sense to append instrumentation data to the
> cobertura.ser file (may need to be done a different way if
> cobertura.ser is a serialized object(s)) and then process all this
> data at reporting time to aggregate it into class statistics?
>=20
> In other words for each test, when it reads in cobertura.ser and
> writes it, it creates its own data either appended or in a separate
> file, and these are all then merged at reporting time.  This would
> reduce disk access and speed up the testing phase and move some of the
> overhead to when the reporting is done.
>=20
> I suppose one possibility is to make use of different cobertura.ser
> files for each module and merge them at the end?  Would appreciate any
> comments you might have.
>=20
> Nathan Johns
>=20
>=20
> -------------------------------------------------------
> This SF.Net email is sponsored by: NEC IT Guy Games.  How far can you sho=
tput
> a projector? How fast can you ride your desk chair down the office luge t=
rack?
> If you want to score the big prize, get to know the little guy.
> Play to win an NEC 61" plasma display: http://www.necitguy.com/?r
> _______________________________________________
> Cobertura-devel mailing list
> Cob...@li...
> https://lists.sourceforge.net/lists/listinfo/cobertura-devel
>