[ 
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)

Reply via email to