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

HDFS-3615. Two BlockTokenSecretManager findbugs warnings. Contributed by Aaron T. Myers. 

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@1360255 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Aaron Myers 2012-07-11 16:09:18 +00:00
parent 9c87911c4a
commit 664ca6b75e
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
View File

@ -450,6 +450,8 @@ Branch-2 ( Unreleased changes )
HDFS-3548. NamenodeFsck.copyBlock fails to create a Block Reader.
(Colin Patrick McCabe via eli)
HDFS-3615. Two BlockTokenSecretManager findbugs warnings. (atm)
BREAKDOWN OF HDFS-3042 SUBTASKS
HDFS-2185. HDFS portion of ZK-based FailoverController (todd)

8
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/security/token/block/BlockTokenSecretManager.java
View File

@ -116,7 +116,7 @@ public class BlockTokenSecretManager extends
}
@VisibleForTesting
public void setSerialNo(int serialNo) {
public synchronized void setSerialNo(int serialNo) {
this.serialNo = (serialNo & LOW_MASK) | (nnIndex << 31);
}
@ -189,7 +189,7 @@ public class BlockTokenSecretManager extends
* Update block keys if update time > update interval.
* @return true if the keys are updated.
*/
public boolean updateKeys(final long updateTime) throws IOException {
public synchronized boolean updateKeys(final long updateTime) throws IOException {
if (updateTime > keyUpdateInterval) {
return updateKeys();
}
@ -371,7 +371,7 @@ public class BlockTokenSecretManager extends
}
@VisibleForTesting
public void setKeyUpdateIntervalForTesting(long millis) {
public synchronized void setKeyUpdateIntervalForTesting(long millis) {
this.keyUpdateInterval = millis;
}
@ -381,7 +381,7 @@ public class BlockTokenSecretManager extends
}
@VisibleForTesting
public int getSerialNoForTesting() {
public synchronized int getSerialNoForTesting() {
return serialNo;
}