Review feedback.

apache · Aug 6, 2014 · cc84b89 · cc84b89
1 parent c1a257a
commit cc84b89
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/yarn/common/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala b/yarn/common/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala
@@ -163,7 +163,7 @@ object YarnSparkHadoopUtil {
         arg.charAt(i) match {
           case '$' => escaped.append("\\$")
           case '"' => escaped.append("\\\"")
-          case '\'' => escaped.append("'\\\"'\\\"'")
+          case '\'' => escaped.append("'\\''")
           case c => escaped.append(c)
         }
       }

diff --git a/yarn/common/src/test/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtilSuite.scala b/yarn/common/src/test/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtilSuite.scala
@@ -36,7 +36,7 @@ class YarnSparkHadoopUtilSuite extends FunSuite with Matchers with Logging {
     }
 
   if (!hasBash) {
-    logInfo("Cannot execute bash, skipping all tests.")
+    logWarning("Cannot execute bash, skipping bash tests.")
   }
 
   def bashTest(name: String)(fn: => Unit) =