Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/1163#discussion_r174433987
--- 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. Please `configuration` move to the next line.
2. You indicating that we upgrade dependency but in fact we added new?
---
