Hi Willem,I managed to fix it from SMX side to honor async processor changes in CAMEL 2.5, so we needn't a new camel release.
SMX4-711[1] to track this issue.
Would you please revert your change with SMX4-707? [1]https://issues.apache.org/jira/browse/SMX4-711 Thanks Freeman On 2010-12-15, at 下午10:07, Willem Jiang wrote:
On 12/15/10 8:27 PM, Gert Vanthienen wrote:These issues[1][2] are related the CXFProducer async process method, I think the unit tests are common use case in ServiceMix.Willem, Personally, I'd rather stick to the 2.5.x version to avoid having to go through any additional dependency upgrades. Is this issue that'sfixed in Camel something that will affect our users or is it somethingthat only pops up in unit tests? What's the Camel issue number for this? If it's a test-only thing or something minor that we can just add to the release notes, we could just avoid running those tests for now, but otherwise it would be best to check with the Camel community if they're open to doing a 2.5.1 release.[1]https://issues.apache.org/jira/browse/CAMEL-3426 [2]https://issues.apache.org/jira/browse/CAMEL-3431I will start a discussion thread in Camel community for the 2.5.1 release.As soon as ServiceMix 4.3.0 is out, we definitely want to move ahead again and pick up a newer SNAPSHOT of Camel including the upgrade for CXF, but I'd really love to get 4.3.0 out first if possible. Regards, Gert Vanthienen ------------------------ FuseSource Web: http://fusesource.com Blog: http://gertvanthienen.blogspot.com/On Wed, Dec 15, 2010 at 10:23 AM, Willem Jiang<[email protected]> wrote:The issue[1] is introduced by changes of camel-cxf component in CAMEL 2.5.0, and I don't find a way to work around the issue in the camel-nmr component, so we need to let servicemix to pick up new version of camel with the patchof the issue.ASFIK, ServiceMix is reusing the Camel features, so it should pick the same version of CXF as the Camel does to avoid to introduce two version of CXF.If we move to Camel 2.6.0 we will introduce CXF 2.3.x which means servicemix need to ship bunch of new bundle of JAXWS 2.2 and JAXB 2.2, etc. It's painful but we still need to move to new version of CXF some time later.As the ServiceMix 4.3.0 is coming out very soon, the other solution isreleasing Camel 2.5.1 to pick up the fix and lock the CXF version. Any thoughts? [1] https://issues.apache.org/jira/browse/SMX4-707 Willem On 12/15/10 4:40 PM, Gert Vanthienen wrote:L.S.,We are working towards a ServiceMix 4.3.0 release. Unless there's a real reason to upgrade to 2.6-SNAPSHOT for Camel, I'd rather keep the Camel 2.5.0 version to avoid having to wait for a new Camel release.Regards, Gert Vanthienen ------------------------ FuseSource Web: http://fusesource.com Blog: http://gertvanthienen.blogspot.com/ On Wed, Dec 15, 2010 at 9:18 AM,<[email protected]> wrote:Author: ningjiang Date: Wed Dec 15 08:18:46 2010 New Revision: 1049452 URL: http://svn.apache.org/viewvc?rev=1049452&view=rev Log: SMX4-707 Fix the camel-nmr failed unit tests Modified:servicemix/smx4/features/trunk/camel/servicemix-camel/src/test/ java/org/apache/servicemix/camel/nmr/ExceptionHandleTest.javaservicemix/smx4/features/trunk/camel/servicemix-camel/src/test/ resources/log4j.propertiesservicemix/smx4/features/trunk/pom.xml Modified:servicemix/smx4/features/trunk/camel/servicemix-camel/src/test/ java/org/apache/servicemix/camel/nmr/ExceptionHandleTest.javaURL: http://svn.apache.org/viewvc/servicemix/smx4/features/trunk/camel/servicemix-camel/src/test/java/org/apache/servicemix/camel/nmr/ExceptionHandleTest.java?rev=1049452&r1=1049451&r2=1049452&view=diff= = = = = = = = = = = = ==================================================================---servicemix/smx4/features/trunk/camel/servicemix-camel/src/test/ java/org/apache/servicemix/camel/nmr/ExceptionHandleTest.java(original) +++servicemix/smx4/features/trunk/camel/servicemix-camel/src/test/ java/org/apache/servicemix/camel/nmr/ExceptionHandleTest.javaWed Dec 15 08:18:46 2010 @@ -163,6 +163,9 @@ public class ExceptionHandleTest extends ClientProxy.getClient(greeter).getInInterceptors().add(new LoggingInInterceptor());ClientProxy.getClient(greeter).getOutInterceptors().add(newLoggingOutInterceptor()); greeter.greetMeOneWay("test oneway"); + // Need to sleep a while as Camel is using Async Engine, + // we need to make sure the camel context is not shutdown rightly. + Thread.sleep(1000); }public void testGetTransportFactoryFromBus() throws Exception {Modified:servicemix/smx4/features/trunk/camel/servicemix-camel/src/test/ resources/log4j.propertiesURL: http://svn.apache.org/viewvc/servicemix/smx4/features/trunk/camel/servicemix-camel/src/test/resources/log4j.properties?rev=1049452&r1=1049451&r2=1049452&view=diff= = = = = = = = = = = = ==================================================================---servicemix/smx4/features/trunk/camel/servicemix-camel/src/test/ resources/log4j.properties(original) +++servicemix/smx4/features/trunk/camel/servicemix-camel/src/test/ resources/log4j.propertiesWed Dec 15 08:18:46 2010 @@ -18,11 +18,11 @@ # # The logging properties used during tests.. # -log4j.rootLogger=DEBUG, out, stdout +log4j.rootLogger=INFO, out, stdout# Separate loggers for Camel and ServiceMix to reduce lock contention-log4j.org.apache.camel=DEBUG, out -log4j.org.apache.servicemix=DEBUG,out +log4j.org.apache.camel=INFO, out +#log4j.org.apache.servicemix=DEBUG,out # CONSOLE appender not used by default log4j.appender.stdout=org.apache.log4j.ConsoleAppender Modified: servicemix/smx4/features/trunk/pom.xml URL: http://svn.apache.org/viewvc/servicemix/smx4/features/trunk/pom.xml?rev=1049452&r1=1049451&r2=1049452&view=diff= = = = = = = = = = = = ==================================================================--- servicemix/smx4/features/trunk/pom.xml (original)+++ servicemix/smx4/features/trunk/pom.xml Wed Dec 15 08:18:46 2010@@ -128,7 +128,7 @@ <activemq.version>5.4.1</activemq.version> <!-- Camel --> -<camel.version>2.5.0</camel.version> +<camel.version>2.6-SNAPSHOT</camel.version> <!-- CXF --> <cxf.version>2.2.11</cxf.version>-- Willem ---------------------------------- FuseSource Web: http://www.fusesource.com Blog: http://willemjiang.blogspot.com (English) http://jnn.javaeye.com (Chinese) Twitter: willemjiang-- Willem ---------------------------------- FuseSource Web: http://www.fusesource.com Blog: http://willemjiang.blogspot.com (English) http://jnn.javaeye.com (Chinese) Twitter: willemjiang
-- Freeman Fang ------------------------ FuseSource: http://fusesource.com blog: http://freemanfang.blogspot.com twitter: http://twitter.com/freemanfang Apache Servicemix:http://servicemix.apache.org Apache Cxf: http://cxf.apache.org Apache Karaf: http://karaf.apache.org Apache Felix: http://felix.apache.org
