Simplify checks

apache · Sep 25, 2014 · c5400cd · c5400cd
1 parent e30ffac
commit c5400cd
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientArguments.scala b/yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientArguments.scala
@@ -69,12 +69,9 @@ private[spark] class ClientArguments(args: Array[String], sparkConf: SparkConf)
    * This is intended to be called only after the provided arguments have been parsed.
    */
   private def validateArgs(): Unit = {
-    Map[Boolean, String](
-      (numExecutors <= 0) -> "You must specify at least 1 executor!"
-    ).foreach { case (errorCondition, errorMessage) =>
-      if (errorCondition) {
-        throw new IllegalArgumentException(errorMessage + "\n" + getUsageMessage())
-      }
+    if (numExecutors <= 0) {
+      throw new IllegalArgumentException(
+        "You must specify at least 1 executor!\n" + getUsageMessage())
     }
   }