Switch back to try to capture things besinds non-zero exit values

Author: holdenk

core/src/test/scala/org/apache/spark/rdd/PipedRDDSuite.scala

@@ -22,6 +22,7 @@ import java.io.File
 import scala.collection.Map
 import scala.io.Codec
 import scala.sys.process._
+import scala.util.Try
 
 import org.apache.hadoop.fs.Path
 import org.apache.hadoop.io.{LongWritable, Text}
@@ -213,7 +214,8 @@ class PipedRDDSuite extends SparkFunSuite with SharedSparkContext {
   }
 
   def testCommandAvailable(command: String): Boolean = {
-    Process(command).run().exitValue() == 0
+    val attempt = Try(Process(command).run().exitValue())
+    attempt.isSuccess && attempt.get == 0
   }
 
   def testExportInputFile(varName: String) {

sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala

@@ -20,6 +20,7 @@ package org.apache.spark.sql.hive.execution
 import java.sql.{Date, Timestamp}
 
 import scala.sys.process.Process
+import scala.util.Try
 
 import org.apache.hadoop.fs.Path
 
@@ -1773,6 +1774,7 @@ class SQLQuerySuite extends QueryTest with SQLTestUtils with TestHiveSingleton {
   }
 
   def testCommandAvailable(command: String): Boolean = {
-    Process(command).run().exitValue() == 0
+    val attempt = Try(Process(command).run().exitValue())
+    attempt.isSuccess && attempt.get == 0
   }
 }