Evgeny Mandrikov

However additional note: line from stack trace ``` at org.jacoco.agent.rt_mlrowg.CoverageTransformer.transform(CoverageTransformer.java:78) ``` maps to valid code in commits between 9daf4f92c339d39a9e5485ff5e9aa9af06e5559c and b05619921551903a81b242b41d196b41c70e770f and corresponds to ``` logger.logException(ex); ``` e.g. https://github.com/jacoco/jacoco/blob/v0.4.1/org.jacoco.agent.rt/src/org/jacoco/agent/rt/CoverageTransformer.java#L78

Might be related to https://bugs.openjdk.java.net/browse/JDK-8140477 , which contains traces of JaCoCo version 0.7.4.

@bjkail Yeah, As usual - thank you very much for your valuable comments. Just some thoughts, guys, about reproducer: So that seems that to trigger loading of classes on a...

@bjkail as of today we support quite a huge range - from dinosaur JDK 5 up to bleeding edge JDK 9 EA 😉 As this is about race condition, reproducer...

@marchof wow, wasn't aware that they worked out fix. I can't find branch, or you meant patch attached to http://forge.ow2.org/tracker/?func=detail&aid=317551&group_id=23&atid=100023 ? Can try to give you a hand in testing...

@bjkail I didn't yet looked inside, so maybe I'm doing something wrong, but seems that it doesn't work for me: ``` $ sed '/Override/d' src/jacoco54/Main.java -i.bak $ java-select 1.5 java...

@bjkail my bad, should have read your comments inside ``` java -verbose:class -javaagent:jacocoagent-0.7.6.jar=excludes=jacoco54.Main -cp src jacoco54.Main ``` perfectly reproduces the problem for me :thumbsup: Thank you!

As was pointed out by @bjkail - set of classes for preloading should include not only our classes, but also JVM classes reachable from ours, so that it can't be...

@bjkail `Preloader2` supposed to behave the same way as `Preloader`, as far as I can see invocations of `Remapper.map`are the same as invocations of `Preloader.addType`, so just a kind of...

Weird, but ``` Instrumenter instrumenter = new Instrumenter(runtime); instrumenter.instrument(new ClassReader(PreMain.class.getName())); ``` in `PreMain` avoids deadlock, while according to `-verbose:class` less classes are loaded in this case (subset) compared to `Preloader`.