Re: svn commit: r1163705 - in /camel/trunk/camel-core/src: main/java/org/apache/camel/impl/converter/PropertyEditorTypeConverter.java test/java/org/apache/camel/component/bean/BeanComponentCustomCreat

2011-09-02 Thread Claus Ibsen
On Thu, Sep 1, 2011 at 3:16 AM, Hadrian Zbarcea hzbar...@gmail.com wrote: Yes, I agree with that, reason why Registries that use caches *are* and should stay as Services. The PropertyEditorTypeConverter in particular doesn't need to be a Service, but the EndpointRegisty, ConsumerCache and

Re: svn commit: r1163705 - in /camel/trunk/camel-core/src: main/java/org/apache/camel/impl/converter/PropertyEditorTypeConverter.java test/java/org/apache/camel/component/bean/BeanComponentCustomCreat

2011-08-31 Thread Claus Ibsen
I wonder if there wont be a problem when you shutdown camel, or run Camel in a hot deployment environment. As before the service would ensure start|stop callbacks, where we could clear the caches when stopping. That is now gone, which means we can just hope the GC eventually will be able to

Re: svn commit: r1163705 - in /camel/trunk/camel-core/src: main/java/org/apache/camel/impl/converter/PropertyEditorTypeConverter.java test/java/org/apache/camel/component/bean/BeanComponentCustomCreat

2011-08-31 Thread Hadrian Zbarcea
Yes, I agree with that, reason why Registries that use caches *are* and should stay as Services. The PropertyEditorTypeConverter in particular doesn't need to be a Service, but the EndpointRegisty, ConsumerCache and ProducerCache should be and are Services. So you are correct about not