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 2017/03/24 22:23:39 UTC

svn commit: r1788552 - in /jmeter/trunk: src/core/org/apache/jmeter/report/processor/graph/AbstractVersusRequestsGraphConsumer.java xdocs/changes.xml

Author: pmouawad
Date: Fri Mar 24 22:23:39 2017
New Revision: 1788552

URL: http://svn.apache.org/viewvc?rev=1788552&view=rev
Log:
Bug 60919 - Report / Dashboard : Latency Vs Request and Response Time Vs Request are wrong if granularity is different from 1000 (1 second)
Bugzilla Id: 60919

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/report/processor/graph/AbstractVersusRequestsGraphConsumer.java
    jmeter/trunk/xdocs/changes.xml

Modified: jmeter/trunk/src/core/org/apache/jmeter/report/processor/graph/AbstractVersusRequestsGraphConsumer.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/report/processor/graph/AbstractVersusRequestsGraphConsumer.java?rev=1788552&r1=1788551&r2=1788552&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/report/processor/graph/AbstractVersusRequestsGraphConsumer.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/report/processor/graph/AbstractVersusRequestsGraphConsumer.java Fri Mar 24 22:23:39 2017
@@ -335,10 +335,11 @@ public abstract class AbstractVersusRequ
                     while (reader.hasNext()) {
                         Sample sample = reader.readSample();
                         // Ask parent to consume the altered sample
+                        Long requestsPerGranularity = counts.get(getTimeInterval(sample)).longValue()
+                                % parent.getGranularity();
+                        Long requestsPerSecond = requestsPerGranularity * 1000 / parent.getGranularity();
                         parent.consumeBase(
-                                createIndexedSample(sample, i,
-                                        counts.get(getTimeInterval(sample)).longValue()
-                                                % parent.getGranularity()), i);
+                                createIndexedSample(sample, i, requestsPerSecond), i);
                     }
                 } finally {
                     file.delete();

Modified: jmeter/trunk/xdocs/changes.xml
URL: http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes.xml?rev=1788552&r1=1788551&r2=1788552&view=diff
==============================================================================
--- jmeter/trunk/xdocs/changes.xml [utf-8] (original)
+++ jmeter/trunk/xdocs/changes.xml [utf-8] Fri Mar 24 22:23:39 2017
@@ -399,6 +399,7 @@ listeners hold and a rework of the way G
 <h3>Report / Dashboard</h3>
 <ul>
     <li><bug>60726</bug>Report / Dashboard : Top 5 errors by samplers must not take into account the series filtering</li>
+    <li><bug>60919</bug>Report / Dashboard : Latency Vs Request and Response Time Vs Request are wrong if granularity is different from 1000 (1 second)</li>
 </ul>
     
 <h3>General</h3>