[ 
https://issues.apache.org/jira/browse/HDFS-5548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13831710#comment-13831710
 ] 

Brandon Li commented on HDFS-5548:
----------------------------------

The patch looks good in general. For the removed log traces, we may want to 
keep them for the sake of debugging. 

> Use ConcurrentHashMap in portmap
> --------------------------------
>
>                 Key: HDFS-5548
>                 URL: https://issues.apache.org/jira/browse/HDFS-5548
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: Haohui Mai
>            Assignee: Haohui Mai
>         Attachments: HDFS-5548.000.patch, HDFS-5548.001.patch
>
>
> Portmap uses a HashMap to store the port mapping. It synchronizes the access 
> of the hash map by locking itself. It can be simplified by using a 
> ConcurrentHashMap.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to