gnodet commented on PR #20958:
URL: https://github.com/apache/camel/pull/20958#issuecomment-3789183695
**Potential concerns:**
1. Closing marker - `}init$` feels a bit awkward compared to alternatives
like `}$init` or `$end$` or even just matching braces with context
2. Variable prefix inconsistency - Variables are declared as `$varname` but
can also be accessed as `${variable.varname}` (two syntaxes for the same thing)
3. Scope persistence - Variables are stored on the Exchange as real
variables, which may have unintended side effects if reused later in the route
(as shown in the docs example with `${variable.level}` in a log statement)
--
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]
