[GitHub] [camel-quarkus] JiriOndrusek commented on a change in pull request #1379: Log message for FileNotFound exception not clear enough #1365

[GitBox <gi...@apache.org>]

JiriOndrusek commented on a change in pull request #1379:
URL: https://github.com/apache/camel-quarkus/pull/1379#discussion_r442491599



##########
File path: extensions-core/core/runtime/src/main/java/org/apache/camel/quarkus/core/CamelRoutesCollector.java
##########
@@ -76,7 +76,7 @@ public XMLRoutesDefinitionLoader getXmlRoutesLoader() {
                     }
                 }
             } catch (FileNotFoundException e) {
-                LOGGER.debug("No XML routes found in {}. Skipping XML routes detection.", part);
+                LOGGER.warn("Resource {} can not be parsed. Skipping XML routes detection.", part);

Review comment:
       Yes I agree. I've overcombined it all. I'll change it to "File {} can not be found".
   This will work in a case that the file doesn't exist or for example if target resource is a folder.




----------------------------------------------------------------
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:
users@infra.apache.org