hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Colin Patrick McCabe (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-7261) storageMap is accessed without synchronization in DatanodeDescriptor#updateHeartbeatState()
Date Mon, 30 Mar 2015 17:48:56 GMT

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

Colin Patrick McCabe commented on HDFS-7261:
--------------------------------------------

+1.  Thanks, [~brahmareddy].

> storageMap is accessed without synchronization in DatanodeDescriptor#updateHeartbeatState()
> -------------------------------------------------------------------------------------------
>
>                 Key: HDFS-7261
>                 URL: https://issues.apache.org/jira/browse/HDFS-7261
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Brahma Reddy Battula
>         Attachments: HDFS-7261-001.patch, HDFS-7261-002.patch, HDFS-7261.patch
>
>
> Here is the code:
> {code}
>       failedStorageInfos = new HashSet<DatanodeStorageInfo>(
>           storageMap.values());
> {code}
> In other places, the lock on "DatanodeDescriptor.storageMap" is held:
> {code}
>     synchronized (storageMap) {
>       final Collection<DatanodeStorageInfo> storages = storageMap.values();
>       return storages.toArray(new DatanodeStorageInfo[storages.size()]);
>     }
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message