Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1207#discussion_r181197234
--- 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 --
Is this necessary? Should not `apache-rat:check` be already bound to the
`validate` phase? ---
