Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2639#discussion_r187484538
--- 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 --
We should not increase the number, decrease or no change is only allowed.
Please fix checkstyle violations based on checkstyle report and keep the number.
---
