Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2433#discussion_r152893622
--- Diff: storm-server/pom.xml ---
@@ -130,7 +130,7 @@
<artifactId>maven-checkstyle-plugin</artifactId>
<!--Note - the version would be inherited-->
<configuration>
- <maxAllowedViolations>2655</maxAllowedViolations>
+ <maxAllowedViolations>2800</maxAllowedViolations>
--- End diff --
No, that means newly added patch should not violate any checkstyle issue.
Even better if we could reduce the number.
---
