Hi Also you could try on your end, by removing that continue code at ProcessorDefinitionHelpe line 253 https://github.com/apache/camel/commit/b08edf391bddb99078f15966394ac264440ffce9#diff-4cb94818eb1df1901b1c8278bd278ba7R253
And do a local build and test with your use-case On Wed, Jul 16, 2014 at 9:16 AM, Claus Ibsen <claus.ib...@gmail.com> wrote: > Yes and as I said, you are welcome to provide an unit test / sample > project / github project or whatever that reproduces the issue. > That helps us fix the issue. > > > > On Wed, Jul 16, 2014 at 9:11 AM, Matt Raible <m...@raibledesigns.com> wrote: >> I believe you. >> >> However, it seems it also creates an issue when using the Java DSL. At least >> from a testing perspective. >> >>> On Jul 16, 2014, at 1:04, Claus Ibsen <claus.ib...@gmail.com> wrote: >>> >>> 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/ > > > > -- > 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/
