luigidemasi commented on PR #27079: URL: https://github.com/apache/camel/pull/27079#issuecomment-5908629496
Addressed the remaining points from [this review](https://github.com/apache/camel/pull/27079#pullrequestreview-5363565737) in [7147494eb882](https://github.com/apache/camel/commit/7147494eb882b952a6c41ab24cfaa90ebbc32b9b): - Merged main and kept its CAMEL-25079 YAML number-quoting implementation, retaining the `needsQuoting("001")` regression test. - Made both `getOtherwiseDefinition()` and `SwitchCaseDefinition.getToDefinition()` read-only. Node preparation is explicit, and regression tests cover prepared IDs/URIs, fallback discovery before startup, independent copies, and fallback-ID collisions regardless of DSL call order. - Confirmed on the [original thread](https://github.com/apache/camel/pull/27079#discussion_r4143208747) that `otherwise` remains an endpoint element so the fallback supports YAML `uri` plus `parameters`. Matching and error-handling behavior are unchanged. Validation passed: 341 focused/exporter tests, 1,085 Java 17 catalog tests against the final regenerated catalog, and the full 695-module clean build with `-DskipTests`. _Comment written by Codex on behalf of @luigidemasi._ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
