Repository: spark
Updated Branches:
  refs/heads/master 1afcf773d -> 37fac1dcd


[SPARK-6477][Build]: Run MIMA tests before the Spark test suite

This moves the MIMA checks to before the full Spark test suite such that, if 
new PR's fail the MIMA check, they will return much faster having not run the 
entire test suite. This is preferable to the current scenario where a user 
would have to wait until the entire test suite completes before realizing it 
failed on a MIMA check in which case, once the MIMA issues are fixed, the user 
would have to resubmit and rerun the full test suite again.

Author: Brennon York <brennon.y...@capitalone.com>

Closes #5145 from brennonyork/SPARK-6477 and squashes the following commits:

12b0aee [Brennon York] updated to put the mima checks before the spark test 
suite


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/37fac1dc
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/37fac1dc
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/37fac1dc

Branch: refs/heads/master
Commit: 37fac1dcd2b0f2845d2952a417fcf85d40351f57
Parents: 1afcf77
Author: Brennon York <brennon.y...@capitalone.com>
Authored: Tue Mar 24 10:33:04 2015 +0000
Committer: Sean Owen <so...@cloudera.com>
Committed: Tue Mar 24 10:33:04 2015 +0000

----------------------------------------------------------------------
 dev/run-tests          | 18 +++++++++---------
 dev/run-tests-codes.sh |  6 +++---
 dev/run-tests-jenkins  |  4 ++--
 3 files changed, 14 insertions(+), 14 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/37fac1dc/dev/run-tests
----------------------------------------------------------------------
diff --git a/dev/run-tests b/dev/run-tests
index d6935a6..561d7fc 100755
--- a/dev/run-tests
+++ b/dev/run-tests
@@ -180,6 +180,15 @@ CURRENT_BLOCK=$BLOCK_BUILD
 
 echo ""
 echo 
"========================================================================="
+echo "Detecting binary incompatibilities with MiMa"
+echo 
"========================================================================="
+
+CURRENT_BLOCK=$BLOCK_MIMA
+
+./dev/mima
+
+echo ""
+echo 
"========================================================================="
 echo "Running Spark unit tests"
 echo 
"========================================================================="
 
@@ -227,12 +236,3 @@ echo 
"========================================================================="
 CURRENT_BLOCK=$BLOCK_PYSPARK_UNIT_TESTS
 
 ./python/run-tests
-
-echo ""
-echo 
"========================================================================="
-echo "Detecting binary incompatibilities with MiMa"
-echo 
"========================================================================="
-
-CURRENT_BLOCK=$BLOCK_MIMA
-
-./dev/mima

http://git-wip-us.apache.org/repos/asf/spark/blob/37fac1dc/dev/run-tests-codes.sh
----------------------------------------------------------------------
diff --git a/dev/run-tests-codes.sh b/dev/run-tests-codes.sh
index 1348e06..8ab6db6 100644
--- a/dev/run-tests-codes.sh
+++ b/dev/run-tests-codes.sh
@@ -22,6 +22,6 @@ readonly BLOCK_RAT=11
 readonly BLOCK_SCALA_STYLE=12
 readonly BLOCK_PYTHON_STYLE=13
 readonly BLOCK_BUILD=14
-readonly BLOCK_SPARK_UNIT_TESTS=15
-readonly BLOCK_PYSPARK_UNIT_TESTS=16
-readonly BLOCK_MIMA=17
+readonly BLOCK_MIMA=15
+readonly BLOCK_SPARK_UNIT_TESTS=16
+readonly BLOCK_PYSPARK_UNIT_TESTS=17

http://git-wip-us.apache.org/repos/asf/spark/blob/37fac1dc/dev/run-tests-jenkins
----------------------------------------------------------------------
diff --git a/dev/run-tests-jenkins b/dev/run-tests-jenkins
index 5f4000e..3a937b6 100755
--- a/dev/run-tests-jenkins
+++ b/dev/run-tests-jenkins
@@ -199,12 +199,12 @@ done
       failing_test="Python style tests"
     elif [ "$test_result" -eq "$BLOCK_BUILD" ]; then
       failing_test="to build"
+    elif [ "$test_result" -eq "$BLOCK_MIMA" ]; then
+      failing_test="MiMa tests"
     elif [ "$test_result" -eq "$BLOCK_SPARK_UNIT_TESTS" ]; then
       failing_test="Spark unit tests"
     elif [ "$test_result" -eq "$BLOCK_PYSPARK_UNIT_TESTS" ]; then
       failing_test="PySpark unit tests"
-    elif [ "$test_result" -eq "$BLOCK_MIMA" ]; then
-      failing_test="MiMa tests"
     else
       failing_test="some tests"
     fi


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to