[jira] [Assigned] (CAMEL-25157) camel-java-io - LwModelToJavaDumper writes invalid Java for some routes

["Claus Ibsen (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/CAMEL-25157?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Claus Ibsen reassigned CAMEL-25157:
-----------------------------------

    Assignee: Claus Ibsen

> camel-java-io - LwModelToJavaDumper writes invalid Java for some routes
> -----------------------------------------------------------------------
>
>                 Key: CAMEL-25157
>                 URL: https://issues.apache.org/jira/browse/CAMEL-25157
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-core
>            Reporter: Claus Ibsen
>            Assignee: Claus Ibsen
>            Priority: Major
>
> While prototyping the Java DSL source parser (CAMEL-25148), a round trip of the camel-xml-io test routes (XML -> model -> Java -> parse -> model -> Java) showed that LwModelToJavaDumper writes Java that is not valid, or not what the Java DSL means, for some routes:
> * *Java-built expressions and predicates are lost.* A route built in Java with a value builder or an expression clause, for example {{.when(header("x").isEqualTo("1"))}}, {{.setProperty("p").constant("v")}} or {{.split(body().tokenize(","))}}, is dumped as {{expression("")}} (or the expression's toString, such as {{expression("simple{${body}} contains y")}}).
> * *enrich/pollEnrich options after the expression clause.* {{.enrich("direct:x")}} is dumped as {{.enrich().aggregateOnException(false).constant("direct:x")}}. ExpressionClause has no aggregateOnException, so this does not compile.
> * *setHeaders/setVariables.* They are dumped as {{.setHeaders().setHeader("h1", ...).setHeader("h2", ...)}}. The Java DSL takes varargs, {{setHeaders("h1", simple(...), "h2", xpath(...))}}, so reading this back gives a setHeaders with no headers and then separate setHeader steps.
> * *Durations are unquoted.* A timeout the DSL stored as a duration string is written as {{.timeout(2s0ms)}} (resequencer batch/stream).
> * *Class-typed options are dropped.* {{throwException(IllegalArgumentException.class, "Forced")}}, {{loadBalance().failover(IOException.class, ...)}} and other options the Java DSL keeps as a Class (exceptionClass, exceptionTypes, typeClass, unmarshalType, ...) lose the class when the model came from Java, because the dumper only writes the String name fields.
> Found with the round-trip test of the CAMEL-25148 prototype (core/camel-java-io, RoundTripTest), where these routes are listed as known gaps.
> _Claude Code on behalf of davsclaus_



--
This message was sent by Atlassian Jira
(v8.20.10#820010)