Re: Issue 52044

2011-10-18 Thread sebb
On 18 October 2011 19:50, Philippe Mouawad
p.moua...@ubik-ingenierie.com wrote:
 Hello Sebb, Milamber , All,
 I investigated this issue, particularly this part of issue

 stramge error for JMS Subscriber - 001 - Response message:
 javax.naming.NamingException: Something already bound at Elite_To_MorphoTrak;

 I created a simple test case and tried not to cache Context and it works
 fine.
 and in fact it is due to the fact that we cache Context and use it by many
 threads.
 From this:
 http://download.oracle.com/javase/jndi/tutorial/beyond/misc/sync.html

 I conclude we should not do caching as it is not mandatory that context is
 Thread Safe (and it's not the case for AMQ one).

 Do you remember why Context were cached ? was it because of bad performances
 if not cached ?

I think that was before my time.

 There are 2 solutions for that:

   - We remove caching
   - We add an  option in GUI to let user select if Context will be cached
   or not

Or we cache per thread, e.g. using ThreadLocal.

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



Re: [Bug 52040] Add a toolbar in JMeter main window

2011-10-18 Thread Milamber


Le 16/10/2011 20:41, Philippe Mouawad a ecrit :
 Hello Milamber,
 I vote for it, I find it great :

- we gain one click each time
- I found icons were very meaningful
   

Thanks ;-)


 Some notes (matter of taste :-)):

- - and + buttons seem bit wide no ?
   

Really? Perhaps I must change the text color for same JMeter icon color
theme (kind of purple)?


 Maybe one missing icon for Start no pauses.
   

I can add this, nonetheless this action is useful only for some specific
runs (execution control for a load test?). Are you sure?

Milamber

 Regards
 Philippe

   


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