Github user ilooner commented on a diff in the pull request:
https://github.com/apache/drill/pull/1207#discussion_r181468772
--- Diff: pom.xml ---
@@ -198,6 +200,78 @@
<build>
<plugins>
+ <plugin>
+ <groupId>org.apache.rat</groupId>
+ <artifactId>apache-rat-plugin</artifactId>
+ <version>0.12</version>
+ <executions>
+ <execution>
--- End diff --
You are right, the docs say it is bound to the validate phase by default
http://creadur.apache.org/rat/apache-rat-plugin/check-mojo.html . I also
removed `<phase>validate</phase>` and tested that it work correctly.
---
