Re: [1/2] git commit: ZooKeeperProducer should shutdown the ConnectionManager when it is closed with thanks to Klaus

2013-12-20 Thread Willem Jiang
I think we can release the zookeeper connection in the ZooKeeperEndpoint. BTW, I found the ZookeeperConsumer has the same issue. -- Willem Jiang Red Hat, Inc. Web: http://www.redhat.com Blog: http://willemjiang.blogspot.com(http://willemjiang.blogspot.com/) (English) http://jnn.iteye.com(http:

Re: [1/2] git commit: ZooKeeperProducer should shutdown the ConnectionManager when it is closed with thanks to Klaus

2013-12-20 Thread Claus Ibsen
Ah we usually keep track on the endpoint number of active users with onProducerStarted / onProducerStopped etc. then we can just stop it when the counter hit zero On Fri, Dec 20, 2013 at 12:41 PM, Willem Jiang wrote: > Hi Claus, > > You are right, we may hit this kind of issue if the endpoint is

Re: [1/2] git commit: ZooKeeperProducer should shutdown the ConnectionManager when it is closed with thanks to Klaus

2013-12-20 Thread Willem Jiang
Hi Claus, You are right, we may hit this kind of issue if the endpoint is shared by different consumer or producer. The ZooKeeperManager is bind to endpoint, we should stop it when the endpoint is stopped. I will updated the code for it. -- Willem Jiang Red Hat, Inc. Web: http://www.redhat.

Re: [1/2] git commit: ZooKeeperProducer should shutdown the ConnectionManager when it is closed with thanks to Klaus

2013-12-20 Thread Claus Ibsen
Hi Is there a problem if you have 2 camel routes using the same zookeeper endpoint, and each route have their own zookeeper producer. And now you shutdown one of the routes, so the other route is still running. If the producer now stops a shared zkm, then the running routes will fail. Just wonde