https://issues.apache.org/bugzilla/show_bug.cgi?id=51840

--- Comment #7 from Sebb <s...@apache.org> 2011-09-19 00:58:45 UTC ---
Thanks for finding the bug and the patch.

I made two changes to InitialContextFactory.java:
* when the context is added to the map using putIfAbsent, this might return an
existing context, in which case we need to return that, rather than the one we
just created, or the Map does no correspond with what we have issued.
* simplified the close method - no need to iterate over the map entries when we
only want the values.

URL: http://svn.apache.org/viewvc?rev=1172403&view=rev
Log:
Bug 51840 - JMS : Cache of InitialContext has some issues

Modified:
  
jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/client/InitialContextFactory.java
  
jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/PublisherSampler.java
  
jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/SubscriberSampler.java
   jakarta/jmeter/trunk/xdocs/changes.xml

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@jakarta.apache.org
For additional commands, e-mail: notifications-h...@jakarta.apache.org

Reply via email to