Hi, One other note regarding a comment you put in code of Random.java regarding the use of ThreadLocalRandom: - https://issues.apache.org/bugzilla/show_bug.cgi?id=54453
Unfortunately my tests don't show performance improvement after adding it. This disturbs me Regards On Sun, Oct 19, 2014 at 9:42 PM, Philippe Mouawad < [email protected]> wrote: > Hello Vladimir, > Thanks for contribution. > For 2, 3 you can open a bugzilla and attach patch. > > > I reviewed the patch, look OK for me, but why don't you also synchronize > LogFunction2#execute since you do for LogFunction#execute ? > > Thanks > > On Sat, Oct 18, 2014 at 4:44 PM, Vladimir Sitnikov < > [email protected]> wrote: > >> Thanks for creating a bug id. I've attached the patch. >> >> What is the preferred way to handle small improvements like 2 and 3? >> >> Vladimir Sitnikov >> > > > > -- > Cordialement. > Philippe Mouawad. > > > -- Cordialement. Philippe Mouawad.
