Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2443#discussion_r155643000
--- Diff: sql/storm-sql-core/pom.xml ---
@@ -66,10 +66,34 @@
<artifactId>calcite-core</artifactId>
--- End diff --
I'll see how the number is changed. It might be remain the same if I fixed
checkstyle issue from A and violated from B.
Actually I'm focusing on planning and addressing window aggregation and
join, so TBH I'd like to address the checkstyle issue later altogether.
---