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/03/07 19:04:13 UTC

svn commit: r515667 - /jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/visualizers/RunningSample.java

Author: sebb
Date: Wed Mar  7 10:04:12 2007
New Revision: 515667

URL: http://svn.apache.org/viewvc?view=rev&rev=515667
Log:
addSample() assumed that the timstamp was always at the end

Modified:
    jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/visualizers/RunningSample.java

Modified: jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/visualizers/RunningSample.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/visualizers/RunningSample.java?view=diff&rev=515667&r1=515666&r2=515667
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/visualizers/RunningSample.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/visualizers/RunningSample.java Wed Mar  7 10:04:12 2007
@@ -200,13 +200,17 @@
 		boolean aSuccessFlag = res.isSuccessful();
 
 		counter++;
-		long startTime = res.getTimeStamp() - aTimeInMillis;
+		long startTime = res.getStartTime();
+		long endTime = res.getEndTime();
+		
 		if (firstTime > startTime) {
 			// this is our first sample, set the start time to current timestamp
 			firstTime = startTime;
 		}
-		if (lastTime < res.getTimeStamp()) {
-			lastTime = res.getTimeStamp();
+		
+		// Always update the end time
+		if (lastTime < endTime) {
+			lastTime = endTime;
 		}
 		runningSum += aTimeInMillis;
 
@@ -345,4 +349,4 @@
 		return errorCount;
 	}
 
-} // class RunningSample
+}



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