David Mollitor created HDFS-14798:
-------------------------------------

             Summary: 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


{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-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to