[
https://issues.apache.org/jira/browse/HDFS-3224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13471675#comment-13471675
]
Daryn Sharp commented on HDFS-3224:
-----------------------------------
I'm a bit squeamish that {{Host2NodesMap.getDatanodeByXferAddr(String, int)}}
requires the caller to have the intimate knowledge of how
{{Host2NodesMap#add(DatanodeDescriptor)}} keyed the node in the map. It's
creating a landmine if someone doesn't update both the {{add}} the caller's
lookups in unison - and we're right back to this bug.
Your thoughts Eli?
> Bug in check for DN re-registration with different storage ID
> -------------------------------------------------------------
>
> Key: HDFS-3224
> URL: https://issues.apache.org/jira/browse/HDFS-3224
> Project: Hadoop HDFS
> Issue Type: Bug
> Reporter: Eli Collins
> Assignee: Jason Lowe
> Priority: Minor
> Attachments: HDFS-3224.patch, HDFS-3224.patch
>
>
> DatanodeManager#registerDatanode checks the host to node map using an IP:port
> key, however the map is keyed on IP, so this check will always fail. It's
> performing the check to determine if a DN with the same IP and storage ID has
> already registered, and if so to remove this DN from the map and indicate
> that eg it's no longer hosting these blocks. This bug has been here forever.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
