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's fixed in Camel something that will affect our users or is it something that 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.
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 patch > of 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 is > releasing 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.java >>> >>> servicemix/smx4/features/trunk/camel/servicemix-camel/src/test/resources/log4j.properties >>> servicemix/smx4/features/trunk/pom.xml >>> >>> Modified: >>> servicemix/smx4/features/trunk/camel/servicemix-camel/src/test/java/org/apache/servicemix/camel/nmr/ExceptionHandleTest.java >>> URL: >>> 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.java >>> Wed 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(new >>> LoggingOutInterceptor()); >>> 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.properties >>> URL: >>> 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.properties >>> Wed 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 >
