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 2008/02/17 01:09:51 UTC

svn commit: r628393 - /jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java

Author: sebb
Date: Sat Feb 16 16:09:50 2008
New Revision: 628393

URL: http://svn.apache.org/viewvc?rev=628393&view=rev
Log:
Access to hosts is from different threads so must be synch

Modified:
    jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java

Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java?rev=628393&r1=628392&r2=628393&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java (original)
+++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java Sat Feb 16 16:09:50 2008
@@ -98,9 +98,9 @@
 
 	private boolean inTest = false;
 
-	private static Map files = new HashMap();
+	private static final Map files = new HashMap();
 
-	private Set hosts = new HashSet();
+	private final Set hosts = new HashSet();
 
 	protected boolean isStats = false;
 
@@ -180,7 +180,7 @@
 		setFilenameProperty(f);
 	}
 
-	public void testEnded(String host) {
+	public synchronized void testEnded(String host) {
 		hosts.remove(host);
 		if (hosts.size() == 0) {
 			finalizeFileOutput();
@@ -188,7 +188,7 @@
 		}
 	}
 
-	public void testStarted(String host) {
+	public synchronized void testStarted(String host) {
 		hosts.add(host);
 		try {
 			initializeFileOutput();



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