You are viewing a plain text version of this content. The canonical link for it is here.
Posted to devnull@infra.apache.org by vl...@apache.org on 2019/06/08 18:42:27 UTC

[jmeter] 23/47: Bug 43449 - Statistical Remote mode does not handle Latency

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

vladimirsitnikov pushed a commit to annotated tag v2_3
in repository https://gitbox.apache.org/repos/asf/jmeter.git

commit 7386860b04a9ce6e8ae27334a4843f24c17a4d45
Author: Sebastian Bazley <se...@apache.org>
AuthorDate: Fri Sep 21 19:05:09 2007 +0000

    Bug 43449 - Statistical Remote mode does not handle Latency
    
    git-svn-id: https://svn.apache.org/repos/asf/jakarta/jmeter/branches/rel-2-2@578229 13f79535-47bb-0310-9956-ffa450edef68
    
    Former-commit-id: ef521266418e5fedfb56e6b58f6f2b089bf7a6b1
---
 src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java | 7 +++++++
 xdocs/changes.xml                                                | 1 +
 2 files changed, 8 insertions(+)

diff --git a/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java b/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java
index 1a9403e..a5aff10 100644
--- a/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java
+++ b/src/core/org/apache/jmeter/samplers/StatisticalSampleResult.java
@@ -38,7 +38,11 @@ public class StatisticalSampleResult extends SampleResult implements
     }
     
 	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 @@ public class StatisticalSampleResult extends SampleResult implements
 		    this.setStartTime(Math.min(getStartTime(), res.getStartTime()));
         }
 		this.setEndTime(Math.max(getEndTime(), res.getEndTime()));
+		
+		setLatency(getLatency()+ res.getLatency());
+
 	}
 
 	public long getTime() {
diff --git a/xdocs/changes.xml b/xdocs/changes.xml
index fc0f661..2500dca 100644
--- a/xdocs/changes.xml
+++ b/xdocs/changes.xml
@@ -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>