Author: sebb Date: Tue Jul 22 08:46:58 2008 New Revision: 678779 URL: http://svn.apache.org/viewvc?rev=678779&view=rev Log: Make instance fields final to improve thread safety
Modified: jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/FixedQueueExecutor.java jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/TemporaryQueueExecutor.java Modified: jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/FixedQueueExecutor.java URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/FixedQueueExecutor.java?rev=678779&r1=678778&r2=678779&view=diff ============================================================================== --- jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/FixedQueueExecutor.java (original) +++ jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/FixedQueueExecutor.java Tue Jul 22 08:46:58 2008 @@ -27,17 +27,21 @@ /** * Request/reply executor with a fixed reply queue. <br> + * + * Used by JMS Sampler (Point to Point) + * * Created on: October 28, 2004 * */ public class FixedQueueExecutor implements QueueExecutor { + + private static final Logger log = LoggingManager.getLoggerForClass(); + /** Sender. */ - private QueueSender producer; + private final QueueSender producer; /** Timeout used for waiting on message. */ - private int timeout; - - private static final Logger log = LoggingManager.getLoggerForClass(); + private final int timeout; /** * Constructor. Modified: jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/TemporaryQueueExecutor.java URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/TemporaryQueueExecutor.java?rev=678779&r1=678778&r2=678779&view=diff ============================================================================== --- jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/TemporaryQueueExecutor.java (original) +++ jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/TemporaryQueueExecutor.java Tue Jul 22 08:46:58 2008 @@ -26,13 +26,16 @@ /** * Request/reply executor with a temporary reply queue. <br> + * + * Used by JMS Sampler (Point to Point) + * * Created on: October 28, 2004 * * @version $Revision$ */ public class TemporaryQueueExecutor implements QueueExecutor { /** The sender and receiver. */ - private QueueRequestor requestor; + private final QueueRequestor requestor; /** * Constructor. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]