Re: [HEADS UP] Classloader mixup with Jetty consumers

2012-10-21 Thread Willem jiang
Just one comment from the TCCL setting on the CamelContinuationServletl, the unit test that you added doesn't have the thread switch even we used the async invocation API. So we need to set the TCCL somewhere incase the we need to use the camel application class loader in the other thread. --

Re: [HEADS UP] Classloader mixup with Jetty consumers

2012-10-21 Thread Willem jiang
Hi Raul, The patches are looking good. +1 for back porting them into 2.10.x and 2.9.x. -- Willem Jiang Red Hat, Inc. FuseSource is now part of Red Hat Web: http://www.fusesource.com | http://www.redhat.com Blog: http://willemjiang.blogspot.com (http://willemjiang.blogspot.com/) (English)

Re: [HEADS UP] Classloader mixup with Jetty consumers

2012-10-21 Thread Raul Kripalani
Now also correcting the TCCL on Jetty consumers with disabled continuations: http://svn.apache.org/viewvc?view=revision&revision=1400734. Regards, *Raúl Kripalani* Apache Camel Committer Enterprise Architect, Program Manager, Open Source Integration specialist http://about.me/raulkripalani | http

[HEADS UP] Classloader mixup with Jetty consumers

2012-10-21 Thread Raul Kripalani
Hello, I've come across what I consider a critical bug in camel-jetty, given the installed base of that component. When multiple Camel Contexts expose camel-jetty consumers on the same TCP port, they share the underlying Jetty Connector. As a side effect of this, the route runs with the classload

Re: About Karaf features validation failing after some dependency upgrade

2012-10-21 Thread Babak Vahdat
Mails in the text format makes the stuff unreadable, so that I attach a diff file to this mail about what I've added to the camel-elasticsearch feature to make the validation pass again. Babak Am 21.10.12 21:35 schrieb "Babak Vahdat" unter : >Hi > >After the following dependency upgrades, the k

About Karaf features validation failing after some dependency upgrade

2012-10-21 Thread Babak Vahdat
Hi After the following dependency upgrades, the karaf features.xml does not validate anymore: http://svn.apache.org/viewvc?view=revision&revision=1396866 The validation fails by camel-elasticsearch: ~/dev/workspace/camel/platforms/karaf/features>mvn clean install -Pvalidate ... [WARNING] Failed

Re: adding Bindings to Endpoints...

2012-10-21 Thread Raul Kripalani
Hi James, The idea looks nice. I'm just a bit concerned about whether the concept steps on the toes of the interceptSendToEndpoint DSL to some extent? The interceptSTE DSL only triggers at the dispatch stage, not at the response, so it doesn't _fully wrap_ it. Perhaps we could extend the intercep