Github user arunmahadevan commented on a diff in the pull request:
https://github.com/apache/storm/pull/2639#discussion_r187495978
--- Diff: external/storm-jms/pom.xml ---
@@ -94,7 +94,7 @@
<artifactId>maven-checkstyle-plugin</artifactId>
<!--Note - the version would be inherited-->
<configuration>
- <maxAllowedViolations>63</maxAllowedViolations>
+ <maxAllowedViolations>73</maxAllowedViolations>
--- End diff --
Actually I can see more violations in JMSSpout.java in master branch (43)
vs the patch (2). I am not sure why. Will fix it anyways.
---
