You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2020/07/25 20:15:26 UTC

[jmeter] branch master updated: Bug 64617 - HTML report: In graph Response Time Percentiles Over Time 90, 95, 99th percentile correspond in reality to 0.90, 0.95 and 0.99 percentiles

This is an automated email from the ASF dual-hosted git repository.

pmouawad pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jmeter.git


The following commit(s) were added to refs/heads/master by this push:
     new 5d47043  Bug 64617 - HTML report: In graph Response Time Percentiles Over Time 90,95,99th percentile correspond in reality to 0.90, 0.95 and 0.99 percentiles
5d47043 is described below

commit 5d470437c87f2807af317284a2d9372d03db987a
Author: pmouawad <p....@ubik-ingenierie.com>
AuthorDate: Sat Jul 25 22:14:55 2020 +0200

    Bug 64617 - HTML report: In graph Response Time Percentiles Over Time
    90,95,99th percentile correspond in reality to 0.90, 0.95 and 0.99
    percentiles
---
 .../graph/impl/ResponseTimePercentilesOverTimeGraphConsumer.java     | 5 +----
 xdocs/changes.xml                                                    | 2 ++
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/core/src/main/java/org/apache/jmeter/report/processor/graph/impl/ResponseTimePercentilesOverTimeGraphConsumer.java b/src/core/src/main/java/org/apache/jmeter/report/processor/graph/impl/ResponseTimePercentilesOverTimeGraphConsumer.java
index 2598292..7c636b6 100644
--- a/src/core/src/main/java/org/apache/jmeter/report/processor/graph/impl/ResponseTimePercentilesOverTimeGraphConsumer.java
+++ b/src/core/src/main/java/org/apache/jmeter/report/processor/graph/impl/ResponseTimePercentilesOverTimeGraphConsumer.java
@@ -100,10 +100,7 @@ public class ResponseTimePercentilesOverTimeGraphConsumer
 
     private GroupInfo createPercentileGroupInfo(String propKey, String label) {
         String seriesName = formatPercentile(label);
-        double defaultValue = new BigDecimal(label)
-                .divide(new BigDecimal("100"), 6, RoundingMode.CEILING)
-                .doubleValue();
-
+        double defaultValue = new BigDecimal(label).setScale(2, RoundingMode.CEILING).doubleValue();
         double property = JMeterUtils.getPropDefault(propKey, defaultValue);
         PercentileAggregatorFactory factory = new PercentileAggregatorFactory();
         factory.setPercentileIndex(property);
diff --git a/xdocs/changes.xml b/xdocs/changes.xml
index c7df4ed..6c1f936 100644
--- a/xdocs/changes.xml
+++ b/xdocs/changes.xml
@@ -163,6 +163,7 @@ Summary
 <h3>Report / Dashboard</h3>
 <ul>
     <li><bug>64547</bug>Report/Dashboard: Ensure graphs Response codes per second is not broken by empty response code in SampleResult. Contributed by Ubik Load Pack (https://ubikloadpack.com)</li>
+    <li><bug>64617</bug>HTML report: In graph Response Time Percentiles Over Time 90,95,99th percentile correspond in reality to 0.90, 0.95 and 0.99 percentiles</li>
 </ul>
 
 <h3>Documentation</h3>
@@ -195,6 +196,7 @@ Summary
 </ul>
 <p>We also thank bug reporters who helped us improve JMeter.</p>
 <ul>
+  <li>Hiroyoshi Mitsumori (mitsumori at mis.dev)</li>
 </ul>
 <p>
 Apologies if we have omitted anyone else.