Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1163#discussion_r174533741
--- Diff: pom.xml ---
@@ -259,7 +259,13 @@
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-checkstyle-plugin</artifactId>
<version>2.12.1</version>
- <configuration>
+ <dependencies>
+ <dependency>
+ <groupId>com.puppycrawl.tools</groupId>
+ <artifactId>checkstyle</artifactId>
+ <version>5.9</version>
+ </dependency>
+ </dependencies> <configuration>
--- End diff --
1. thanks for catching
2. each version of `maven-checkstyle-plugin` is preconfigured with a
specific version of checkstyle (2.12.1 uses 5.7) that can be overridden by
specifying that dependency explicitly.
---
