On 19 September 2011 06:45, Philippe Mouawad <p.moua...@ubik-ingenierie.com> wrote: > Hello, > I closed bug as fixed, don't know if I had to do it or it's up to you.
That's OK as you raised the issue. I had intended to close it myself, but forgot. Only a JMeter developer or the originator should normally close issues. > Thank you > Regards > Philippe > > On Monday, September 19, 2011, <bugzi...@apache.org> wrote: >> 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 on the CC list for the bug. >> You reported the bug. >> > > -- > Cordialement. > Philippe Mouawad. > Ubik-Ingénierie > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@jakarta.apache.org For additional commands, e-mail: dev-h...@jakarta.apache.org
