essobedo commented on code in PR #9648:
URL: https://github.com/apache/camel/pull/9648#discussion_r1149195999
##########
components/camel-groovy/src/main/java/org/apache/camel/language/groovy/GroovyShellFactory.java:
##########
@@ -27,4 +29,14 @@ default String getFileName(Exchange exchange) {
return null;
}
+ /**
+ * This method provide a way to define some global variables that will be
applied to all the groovy script context.
+ *
+ * @param exchange the camel exchange in process.
+ * @return the global variables that will be applied to all the groovy
script context.
+ */
+ default Map<String, Object> getVariables(Exchange exchange) {
+ return null;
Review Comment:
Maybe we could return an empty map instead, to avoid having to check if it
is null and then have a code easier to read
--
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]
