Claus Ibsen created CAMEL-25157:
-----------------------------------
Summary: 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
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)