Hi Yeah but the tests in camel-spring would likely fail. That commit fixes some issues when using xml dsl to define routes (spring or blueprint)
On Tue, Jul 15, 2014 at 8:19 PM, Matt Raible <m...@raibledesigns.com> wrote: > FWIW, all tests pass in camel-core with this change reverted. > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 20:57 min > [INFO] Finished at: 2014-07-15T12:18:03-07:00 > [INFO] Final Memory: 25M/376M > > > On Tue, Jul 15, 2014 at 11:01 AM, Matt Raible <m...@raibledesigns.com> > wrote: > >> If I revert the following change to AdviceWithTasks.java, it seems to >> solve this issue: >> >> >> https://github.com/apache/camel/commit/b08edf391bddb99078f15966394ac264440ffce9 >> >> I cloned master, reverted this change, then installed everything to test. >> >> On Jul 15, 2014, at 7:59 AM, Claus Ibsen <claus.ib...@gmail.com> wrote: >> >> > On Tue, Jul 15, 2014 at 3:56 PM, Matt Raible <m...@raibledesigns.com> >> wrote: >> >> It tried it with 2.13.2 and it works just fine. >> >> >> > >> > Thanks. >> > >> > If its possible for you then you are welcome to .zip an unit test / >> > sample project that has this issue and create a JIRA ticket and attach >> > it, or some github repo or something else. >> > >> > I tired to reproduce from your mail this morning but the route worked >> for me. >> > >> >> >> >> On Tue, Jul 15, 2014 at 12:12 AM, Claus Ibsen <claus.ib...@gmail.com> >> wrote: >> >> >> >>> Hi >> >>> >> >>> Can you try with 2.13.2 also? >> >>> >> >>> There was a fix in this to fix an issue when using Content Based >> >>> Router and weaveById. I bet it may has cause a regression for your >> >>> use-case. >> >>> >> >>> On Mon, Jul 14, 2014 at 11:31 PM, Matt Raible <m...@raibledesigns.com> >> >>> wrote: >> >>>> Hey all, >> >>>> >> >>>> I have the following route defined with the Java DSL: >> >>>> >> >>>> from("direct:localMemberLookup").routeId("localMemberLookup") >> >>>> .process(new >> >>> MemberLookupToSqlParametersProcessor()).id("sqlParams") >> >>>> >> >>> .recipientList(simple("sql:{{sql.memberLookup}}")).delimiter("false") >> >>>> .to("log:output") >> >>>> .process(new MemberLookupProcessor()) >> >>>> // do more processing >> >>>> .to("log:output"); >> >>>> >> >>>> I'm testing it with a test that looks as follows: >> >>>> >> >>>> @EndpointInject(uri = "mock:lookupHeaders") >> >>>> MockEndpoint lookupHeaders; >> >>>> >> >>>> @EndpointInject(uri = "mock:searchResult") >> >>>> MockEndpoint searchResult; >> >>>> >> >>>> @EndpointInject(uri = "mock:lookupResult") >> >>>> MockEndpoint lookupResult; >> >>>> >> >>>> @Autowired >> >>>> CamelContext camelContext; >> >>>> >> >>>> @Before >> >>>> public void before() throws Exception { >> >>>> ModelCamelContext context = (ModelCamelContext) camelContext; >> >>>> context.setTracing(true); >> >>>> RouteDefinition searchRoute = >> >>> context.getRouteDefinition("memberSearchRequest"); >> >>>> searchRoute.to(searchResult); >> >>>> >> >>>> RouteDefinition lookupRoute = >> >>> context.getRouteDefinition("localMemberLookup"); >> >>>> lookupRoute.adviceWith(context, new AdviceWithRouteBuilder() { >> >>>> @Override >> >>>> public void configure() throws Exception { >> >>>> >> weaveById("sqlParams").after().to(lookupHeaders); >> >>>> } >> >>>> }); >> >>>> lookupRoute.to(lookupResult); >> >>>> context.start(); >> >>>> } >> >>>> >> >>>> With Camel 2.13.1, this works fine. However, with 2.14-SNAPSHOT, I get >> >>> the following error: >> >>>> >> >>>> java.lang.IllegalArgumentException: There are no outputs which >> matches: >> >>> sqlParams in the route >> >>>> >> >>>> Any ideas? >> >>>> >> >>>> Thanks, >> >>>> >> >>>> Matt >> >>> >> >>> >> >>> >> >>> -- >> >>> Claus Ibsen >> >>> ----------------- >> >>> Red Hat, Inc. >> >>> Email: cib...@redhat.com >> >>> Twitter: davsclaus >> >>> Blog: http://davsclaus.com >> >>> Author of Camel in Action: http://www.manning.com/ibsen >> >>> hawtio: http://hawt.io/ >> >>> fabric8: http://fabric8.io/ >> >>> >> > >> > >> > >> > -- >> > Claus Ibsen >> > ----------------- >> > Red Hat, Inc. >> > Email: cib...@redhat.com >> > Twitter: davsclaus >> > Blog: http://davsclaus.com >> > Author of Camel in Action: http://www.manning.com/ibsen >> > hawtio: http://hawt.io/ >> > fabric8: http://fabric8.io/ >> >> -- Claus Ibsen ----------------- Red Hat, Inc. Email: cib...@redhat.com Twitter: davsclaus Blog: http://davsclaus.com Author of Camel in Action: http://www.manning.com/ibsen hawtio: http://hawt.io/ fabric8: http://fabric8.io/
