elek commented on issue #1435: HDDS-2119. Use checkstyle.xml and suppressions.xml in hdds/ozone projects for checkstyle validation. URL: https://github.com/apache/hadoop/pull/1435#issuecomment-531450551 Thanks @nandakumar131 the patch. Unfortunately it seems to be failing for me with testing it with the checkstyle.sh. Until now the checkstyle.sh did a `mvn checkstyle:check` without compilation (which made it very fast). But now you should have the build-tools compiled and installed to do a full checkstyle check. I don't know what is the good solution here 1. We can use file references instead of artifact references to define the checkstyle rule files 2. Or we can add the required build-tools install to the checkstyle.sh I liked that it was very fast until know, therefore I would prefer the first one, but there could be other options...
---------------------------------------------------------------- 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 With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org