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/05/19 19:04:40 UTC

svn commit: r539777 - /jakarta/jmeter/branches/rel-2-2/src/functions/org/apache/jmeter/functions/CSVRead.java

Author: sebb
Date: Sat May 19 10:04:39 2007
New Revision: 539777

URL: http://svn.apache.org/viewvc?view=rev&rev=539777
Log:
Functions are shared between threads, so setParameters() and execute() need to be synchronized if there are any class fields.

Modified:
    jakarta/jmeter/branches/rel-2-2/src/functions/org/apache/jmeter/functions/CSVRead.java

Modified: jakarta/jmeter/branches/rel-2-2/src/functions/org/apache/jmeter/functions/CSVRead.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/functions/org/apache/jmeter/functions/CSVRead.java?view=diff&rev=539777&r1=539776&r2=539777
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/functions/org/apache/jmeter/functions/CSVRead.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/functions/org/apache/jmeter/functions/CSVRead.java Sat May 19 10:04:39 2007
@@ -145,7 +145,7 @@
 	/**
 	 * @see org.apache.jmeter.functions.Function#setParameters(Collection)
 	 */
-	public void setParameters(Collection parameters) throws InvalidVariableException {
+	public synchronized void setParameters(Collection parameters) throws InvalidVariableException {
 		log.debug("setParameter - Collection.size=" + parameters.size());
 
 		values = parameters.toArray();



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