Re: DO NOT REPLY [Bug 51840] JMS : Cache of InitialContext has some issues

On 19 September 2011 06:45, Philippe Mouawad
<[email protected]> 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,  <[email protected]> wrote:
>> https://issues.apache.org/bugzilla/show_bug.cgi?id=51840
>>
>> --- Comment #7 from Sebb <[email protected]> 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
>