Hello, Any other thoughts ? Thanks
On Saturday, May 6, 2017, Antonio Gomes Rodrigues <[email protected]> wrote: > +1 > > 2017-05-06 15:57 GMT+02:00 Maxime Chassagneux < > [email protected] <javascript:;>> > : > > > +1 : lgtm > > > > 2017-05-06 15:41 GMT+02:00 Philippe Mouawad < > [email protected] <javascript:;> > > >: > > > > > Hello, > > > A user recently reported a bug on StatCalculator class: > > > > > > - https://bz.apache.org/bugzilla/show_bug.cgi?id=61071 > > > > > > It appears issue is not restrained to Median but concerns percentile > > > computing. > > > > > > Besides, this class has many drawbacks: > > > > > > - It is not tested as much as it should be > > > - It relies on an algorithm that consumes memory > > > > > > We use another class in Web/Dashboard report and BackendListener client > > > implementations that relies on commons-math > > > org.apache.commons.math3.stat.descriptive.DescriptiveStatistics. > > > > > > My proposal is the following: > > > > > > - First step : Introduce in StatCalculator, HdrHistogram > > > - Second step : Replace DescriptiveStatistics by HdrHistogram > > > > > > > > > Benefits: > > > > > > - Uniform computing accross JMeter > > > - Better performances > > > > > > Are you ok with this approach ? > > > > > > Thanks > > > -- > > > Regards. > > > Philippe Mouawad. > > > Ubik-Ingénierie > > > > > > -- Cordialement. Philippe Mouawad.
