Repository: hadoop
Updated Branches:
  refs/heads/HDFS-1312 76a1391d5 -> 32058f9b6


HDFS-10496. DiskBalancer: ExecuteCommand checks planFile in a wrong way. 
Contributed by Lei (Eddy) Xu.


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/32058f9b
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/32058f9b
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/32058f9b

Branch: refs/heads/HDFS-1312
Commit: 32058f9b6d49b93ac8e99da383cb86170a16b317
Parents: 76a1391
Author: Anu Engineer <[email protected]>
Authored: Tue Jun 7 15:10:22 2016 -0700
Committer: Anu Engineer <[email protected]>
Committed: Tue Jun 7 15:10:22 2016 -0700

----------------------------------------------------------------------
 .../hadoop/hdfs/server/diskbalancer/command/ExecuteCommand.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/32058f9b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/diskbalancer/command/ExecuteCommand.java
----------------------------------------------------------------------
diff --git 
a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/diskbalancer/command/ExecuteCommand.java
 
b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/diskbalancer/command/ExecuteCommand.java
index c17ef00..6d30e86 100644
--- 
a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/diskbalancer/command/ExecuteCommand.java
+++ 
b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/diskbalancer/command/ExecuteCommand.java
@@ -61,7 +61,7 @@ public class ExecuteCommand extends Command {
     verifyCommandOptions(DiskBalancer.EXECUTE, cmd);
 
     String planFile = cmd.getOptionValue(DiskBalancer.EXECUTE);
-    Preconditions.checkArgument(planFile == null || planFile.isEmpty(),
+    Preconditions.checkArgument(planFile != null && !planFile.isEmpty(),
         "Invalid plan file specified.");
 
     String planData = null;


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to