You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2007/09/21 21:05:13 UTC

svn commit: r578229 - in /jakarta/jmeter: branches/rel-2-2/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java branches/rel-2-2/xdocs/changes.xml trunk/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java trunk/xdocs/changes.xml

Author: sebb
Date: Fri Sep 21 12:05:09 2007
New Revision: 578229

URL: http://svn.apache.org/viewvc?rev=578229&view=rev
Log:
Bug 43449 - Statistical Remote mode does not handle Latency

Modified:
    jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java
    jakarta/jmeter/branches/rel-2-2/xdocs/changes.xml
    jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java
    jakarta/jmeter/trunk/xdocs/changes.xml

Modified: jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java?rev=578229&r1=578228&r2=578229&view=diff
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java Fri Sep 21 12:05:09 2007
@@ -38,7 +38,11 @@
     }
     
 	public StatisticalSampleResult(SampleResult res) {
+		// Copy data that is shared between samples:
 		setSampleLabel(res.getSampleLabel());
+		setThreadName(res.getThreadName());
+		setSuccessful(true); // Assume result is OK
+		setSampleCount(0); // because we add the sample count in later
 	}
 
 	public void add(SampleResult res) {
@@ -59,6 +63,9 @@
 		    this.setStartTime(Math.min(getStartTime(), res.getStartTime()));
         }
 		this.setEndTime(Math.max(getEndTime(), res.getEndTime()));
+		
+		setLatency(getLatency()+ res.getLatency());
+
 	}
 
 	public long getTime() {

Modified: jakarta/jmeter/branches/rel-2-2/xdocs/changes.xml
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/xdocs/changes.xml?rev=578229&r1=578228&r2=578229&view=diff
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/xdocs/changes.xml (original)
+++ jakarta/jmeter/branches/rel-2-2/xdocs/changes.xml Fri Sep 21 12:05:09 2007
@@ -39,6 +39,7 @@
 <li>Bug 42948 - Problems with Proxy gui table fields in Java 1.6</li>
 <li>Fixup broken jmeter-server script</li>
 <li>Bug 43364 - option to revert If Controller to pre 2.3RC3 behaviour</li>
+<li>Bug 43449 - Statistical Remote mode does not handle Latency</li>
 </ul>
 
 <h4>Improvements</h4>

Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java?rev=578229&r1=578228&r2=578229&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java (original)
+++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java Fri Sep 21 12:05:09 2007
@@ -38,7 +38,11 @@
     }
     
 	public StatisticalSampleResult(SampleResult res) {
+		// Copy data that is shared between samples:
 		setSampleLabel(res.getSampleLabel());
+		setThreadName(res.getThreadName());
+		setSuccessful(true); // Assume result is OK
+		setSampleCount(0); // because we add the sample count in later
 	}
 
 	public void add(SampleResult res) {
@@ -59,6 +63,9 @@
 		    this.setStartTime(Math.min(getStartTime(), res.getStartTime()));
         }
 		this.setEndTime(Math.max(getEndTime(), res.getEndTime()));
+		
+		setLatency(getLatency()+ res.getLatency());
+
 	}
 
 	public long getTime() {

Modified: jakarta/jmeter/trunk/xdocs/changes.xml
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/xdocs/changes.xml?rev=578229&r1=578228&r2=578229&view=diff
==============================================================================
--- jakarta/jmeter/trunk/xdocs/changes.xml (original)
+++ jakarta/jmeter/trunk/xdocs/changes.xml Fri Sep 21 12:05:09 2007
@@ -57,6 +57,7 @@
 <li>Bug 42948 - Problems with Proxy gui table fields in Java 1.6</li>
 <li>Fixup broken jmeter-server script</li>
 <li>Bug 43364 - option to revert If Controller to pre 2.3RC3 behaviour</li>
+<li>Bug 43449 - Statistical Remote mode does not handle Latency</li>
 </ul>
 
 <h4>Improvements</h4>



---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org