Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HDFS-11409. DatanodeInfo getNetworkLocation and setNetworkLocation shoud use volatile instead of synchronized. Contributed by Chen Liang. 

(cherry picked from commit aaf2713235)
This commit is contained in:
Xiaoyu Yao 2017-02-14 12:52:34 -08:00
parent 92dbdbc490
commit d72f1c5976
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/DatanodeInfo.java
View File

@ -51,7 +51,7 @@ public class DatanodeInfo extends DatanodeID implements Node {
private long lastUpdate;
private long lastUpdateMonotonic;
private int xceiverCount;
private String location = NetworkTopology.DEFAULT_RACK;
private volatile String location = NetworkTopology.DEFAULT_RACK;
private String softwareVersion;
private List<String> dependentHostNames = new LinkedList<>();
private String upgradeDomain;
@ -292,10 +292,12 @@ public class DatanodeInfo extends DatanodeID implements Node {
}
/** network location */
public synchronized String getNetworkLocation() {return location;}
@Override
public String getNetworkLocation() {return location;}
/** Sets the network location */
public synchronized void setNetworkLocation(String location) {
@Override
public void setNetworkLocation(String location) {
this.location = NodeBase.normalize(location);
}