I don't think it was a bad idea, JMeter evolution should not be stopped by a particular bug in a particular (old) version of Java fixed in a newer version.
Regarding this issue, are you sure the bug is not fixed even in Java7 since 7u2 ? - http://www.oracle.com/technetwork/java/javase/2col/7u2bugfixes-1394661.html Regards Philippe On Mon, Jan 25, 2016 at 10:53 PM, Philippe Mouawad < [email protected]> wrote: > For info > ---------- Forwarded message ---------- > From: vdaburon <[email protected]> > Date: Thu, Jan 21, 2016 at 9:51 PM > Subject: Re: [jmeter] Bug 54453 - Performance enhancements : Replace > Random by ThreadLocalRandom (b969ff4) > To: apache/jmeter <[email protected]> > Cc: Philippe M <[email protected]> > > > Hi, > This modification was not a good idea for jdk 7 because the > ThreadLocalRandom had bugs > http://bugs.java.com/bugdatabase/view_bug.do?bug_id=7051516 > JDK-7051516 : ThreadLocalRandom seed is never initialized so all instances > generate the same sequence > > May you need to check the jdk version 7 or 8 before choose the > ThreadLocalRandom or not ? > > Regards. > Vincent D. > > — > Reply to this email directly or view it on GitHub > <https://github.com/apache/jmeter/commit/b969ff4ed9fdc509c32fc09b95732ed137060139#commitcomment-15597343> > . > > -- Cordialement. Philippe Mouawad. Ubik-Ingénierie UBIK LOAD PACK Web Site <http://www.ubikloadpack.com/> UBIK LOAD PACK on TWITTER <https://twitter.com/ubikloadpack>
