dervan commented on code in PR #2334:
URL: https://github.com/apache/avro/pull/2334#discussion_r1262527249
##########
lang/java/maven-plugin/src/main/java/org/apache/avro/mojo/AbstractAvroMojo.java:
##########
@@ -213,7 +213,9 @@ public void execute() throws MojoExecutionException {
if (hasImports) {
for (String importedFile : imports) {
File file = new File(importedFile);
- if (file.isDirectory()) {
+ if (!file.exists()) {
Review Comment:
Of course I understand that - I just pointed out that _getIncludedFiles_
touches this file earlier, but with no real downside. Anyway, I moved existence
check to the separate method that checks everything in the first place - will
that work for you?
--
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]
