[ https://issues.apache.org/jira/browse/HDFS-7386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14209378#comment-14209378 ]
Yongjun Zhang commented on HDFS-7386: ------------------------------------- Many thanks Chris! I just uploaded rev 002 to address both of your comments. Really appreciate your help on reviewing the patch. I searched the code base for 1024 but not 1023:-) Very nice that you pointed out the place I missed. > Replace check "port number < 1024" with shared isPrivilegedPort method > ----------------------------------------------------------------------- > > Key: HDFS-7386 > URL: https://issues.apache.org/jira/browse/HDFS-7386 > Project: Hadoop HDFS > Issue Type: Improvement > Reporter: Yongjun Zhang > Assignee: Yongjun Zhang > Priority: Trivial > Attachments: HDFS-7386.001.patch, HDFS-7386.002.patch > > > Per discussion in HDFS-7382, I'm filing this jira as a follow-up, to replace > check "port number < 1024" with shared isPrivilegedPort method. > Thanks [~cnauroth] for the work on HDFS-7382 and suggestion there. -- This message was sent by Atlassian JIRA (v6.3.4#6332)