[GitHub] [giraph] atanu1991 commented on a change in pull request #150: GIRAPH-1251: Add SSLHandler for all Netty Communication (Initial diff)

[GitBox]

atanu1991 commented on a change in pull request #150:
URL: https://github.com/apache/giraph/pull/150#discussion_r625430788



##########
File path: checkstyle.xml
##########
@@ -227,7 +227,7 @@
     </module>
       <!-- Over time, we will revised this down -->
     <module name="MethodLength">
-      <property name="max" value="210"/>
+      <property name="max" value="220"/>

Review comment:
       Change needed for maven checkstyle




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org