hadoop-hdfs-issues mailing list archives

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

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

Hudson commented on HDFS-7261:
------------------------------

FAILURE: Integrated in Hadoop-trunk-Commit #7463 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/7463/])
HDFS-7261. storageMap is accessed without synchronization in DatanodeDescriptor#updateHeartbeatState()
(Brahma Reddy Battula via Colin P. McCabe) (cmccabe: rev 1feb9569f366a29ecb43592d71ee21023162c18f)
* hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
* hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeDescriptor.java


> 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