davsclaus commented on code in PR #28257:
URL: https://github.com/apache/camel/pull/28257#discussion_r4344925410
##########
components/camel-jte/src/main/java/org/apache/camel/component/jte/JteCodeResolver.java:
##########
@@ -39,9 +39,9 @@ public JteCodeResolver(CamelContext camelContext) {
this.camelContext = camelContext;
}
- public void addTemplateFromHeader(String exchangeId, String template) {
- if (exchangeId != null && template != null) {
- headerTemplates.put(exchangeId, template);
+ public void addTemplateFromHeader(String name, String template) {
+ if (name != null && template != null) {
+ headerTemplates.put(name, template);
Review Comment:
Optional: with names per content, `resolve` (which removes the entry) is
called only when the engine compiles the name, so after the first exchange this
put leaves the entry in the map. It is one copy per distinct template, so
bounded, but worth a one-line comment here.
--
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]
