Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2443#discussion_r155320991
--- Diff: sql/storm-sql-core/pom.xml ---
@@ -66,10 +66,34 @@
<artifactId>calcite-core</artifactId>
--- End diff --
You might want to reduce the allowed checkstyle errors in the poms if you
reduced the violations.---
