metters commented on a change in pull request #38:
URL: https://github.com/apache/camel-examples/pull/38#discussion_r584075013
##########
File path:
examples/main/src/main/java/org/apache/camel/example/StandaloneCamel.java
##########
@@ -34,29 +34,30 @@ private StandaloneCamel() {
public static void main(String[] args) throws Exception {
// create a new CamelContext
- CamelContext camelContext = new DefaultCamelContext();
+ try (CamelContext camelContext = new DefaultCamelContext()) {
Review comment:
@davsclaus , may I ask, why we ignore that Sonar issue?
I have to admit that I have made this kind of change several times in the
past. Usually not without testing it, therefore only on modules that I was able
to start/run.
However, I deleted my branches and because the modules were moved/renamed it
will be quite difficult to find out where I did those changes (wrapping a
statement in try with resources)
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
