FIX: Don't build Hive in assembly unless running Hive tests.

This will make the tests more stable when not running SQL tests.

Author: Patrick Wendell <pwendell@gmail.com>

Closes #439 from pwendell/hive-tests and squashes the following commits:

88a6032 [Patrick Wendell] FIX: Don't build Hive in assembly unless running Hive tests.

1 files changed, 6 insertions, 7 deletions

diff --git a/dev/run-tests b/dev/run-tests
index 7be58588b1..6043f859ae 100755
--- a/dev/run-tests
+++ b/dev/run-tests
@@ -38,7 +38,7 @@ if [ -n "$AMPLAB_JENKINS" ]; then
   diffs=`git diff --dirstat master | awk '{ print $2; }' | grep "^sql/"`
   if [ -n "$diffs" ]; then
     echo "Detected changes in SQL. Will run Hive test suite."
-    run_sql_tests=true
+    export _RUN_SQL_TESTS=true # exported for PySpark tests
   fi
 fi
 
@@ -62,13 +62,12 @@ echo "========================================================================="
 # echo "q" is needed because sbt on encountering a build file with failure 
 # (either resolution or compilation) prompts the user for input either q, r, 
 # etc to quit or retry. This echo is there to make it not block.
-echo -e "q\n" | SPARK_HIVE=true sbt/sbt clean assembly | \
-  grep -v -e "info.*Resolving" -e "warn.*Merging" -e "info.*Including"
-
-if [ -n "$run_sql_tests" ]; then
-  echo -e "q\n" | SPARK_HIVE=true sbt/sbt test | grep -v -e "info.*Resolving" 
+if [ -n "$_RUN_SQL_TESTS" ]; then
+  echo -e "q\n" | SPARK_HIVE=true sbt/sbt clean assembly test | \
+    grep -v -e "info.*Resolving" -e "warn.*Merging" -e "info.*Including"
 else
-  echo -e "q\n" | sbt/sbt test | grep -v -e "info.*Resolving" 
+  echo -e "q\n" | sbt/sbt clean assembly test | \
+    grep -v -e "info.*Resolving" -e "warn.*Merging" -e "info.*Including"
 fi
 
 echo "========================================================================="