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

2011-09-19 Thread sebb
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=1172403view=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



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

2011-09-18 Thread Philippe Mouawad
Hello,
I closed  bug as fixed, don't know if I had to do it or it's up to you.

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=1172403view=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