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-14798) Synchronize invalidateBlocks in DatanodeDescriptor
Date Thu, 12 Sep 2019 14:21:00 GMT

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

Hudson commented on HDFS-14798:
-------------------------------

FAILURE: Integrated in Jenkins build Hadoop-trunk-Commit #17286 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/17286/])
HDFS-14798. Synchronize invalidateBlocks in DatanodeDescriptor. (surendralilhore: rev 2ff2a7f6120079b6a88afff987a551fa3d1f47e2)
* (edit) hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeDescriptor.java


> Synchronize invalidateBlocks in DatanodeDescriptor
> --------------------------------------------------
>
>                 Key: HDFS-14798
>                 URL: https://issues.apache.org/jira/browse/HDFS-14798
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: namenode
>    Affects Versions: 3.2.0
>            Reporter: David Mollitor
>            Assignee: hemanthboyina
>            Priority: Minor
>              Labels: n00b, newbie
>         Attachments: HDFS-14798.001.patch
>
>
> {code:java|title=DatanodeDescriptor.java}
> public void resetBlocks() {
>   ...
>   this.invalidateBlocks.clear();
>   ...
> }
> public void clearBlockQueues() {
>   synchronized (invalidateBlocks) {
>     this.invalidateBlocks.clear();
>   }
>   ...
> }
> {code}
> It may not be strictly necessary, but why risk it? The invalidateBlocks should be protected
in {{resetBlocks()}} just like it is in {{clearBlockQueues()}}/



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-help@hadoop.apache.org


Mime
View raw message