You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by ma...@apache.org on 2014/09/18 00:14:09 UTC
git commit: [SPARK-3534] Fix expansion of testing arguments to sbt
Repository: spark
Updated Branches:
refs/heads/master b3830b28f -> 7fc3bb7c8
[SPARK-3534] Fix expansion of testing arguments to sbt
Testing arguments to `sbt` need to be passed as an array, not a single, long string.
Fixes a bug introduced in #2420.
Author: Nicholas Chammas <ni...@gmail.com>
Closes #2437 from nchammas/selective-testing and squashes the following commits:
a9f9c1c [Nicholas Chammas] fix printing of sbt test arguments
cf57cbf [Nicholas Chammas] fix sbt test arguments
e33b978 [Nicholas Chammas] Merge pull request #2 from apache/master
0b47ca4 [Nicholas Chammas] Merge branch 'master' of github.com:nchammas/spark
8051486 [Nicholas Chammas] Merge pull request #1 from apache/master
03180a4 [Nicholas Chammas] Merge branch 'master' of github.com:nchammas/spark
d4c5f43 [Nicholas Chammas] Merge pull request #6 from apache/master
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/7fc3bb7c
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/7fc3bb7c
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/7fc3bb7c
Branch: refs/heads/master
Commit: 7fc3bb7c88a6bf5348d52ffee37a220a47c5a398
Parents: b3830b2
Author: Nicholas Chammas <ni...@gmail.com>
Authored: Wed Sep 17 15:14:04 2014 -0700
Committer: Michael Armbrust <mi...@databricks.com>
Committed: Wed Sep 17 15:14:04 2014 -0700
----------------------------------------------------------------------
dev/run-tests | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/spark/blob/7fc3bb7c/dev/run-tests
----------------------------------------------------------------------
diff --git a/dev/run-tests b/dev/run-tests
index 53148d2..7c00216 100755
--- a/dev/run-tests
+++ b/dev/run-tests
@@ -141,17 +141,20 @@ echo "========================================================================="
{
# If the Spark SQL tests are enabled, run the tests with the Hive profiles enabled.
+ # This must be a single argument, as it is.
if [ -n "$_RUN_SQL_TESTS" ]; then
SBT_MAVEN_PROFILES_ARGS="$SBT_MAVEN_PROFILES_ARGS -Phive"
fi
if [ -n "$_SQL_TESTS_ONLY" ]; then
- SBT_MAVEN_TEST_ARGS="catalyst/test sql/test hive/test"
+ # This must be an array of individual arguments. Otherwise, having one long string
+ #+ will be interpreted as a single test, which doesn't work.
+ SBT_MAVEN_TEST_ARGS=("catalyst/test" "sql/test" "hive/test")
else
- SBT_MAVEN_TEST_ARGS="test"
+ SBT_MAVEN_TEST_ARGS=("test")
fi
- echo "[info] Running Spark tests with these arguments: $SBT_MAVEN_PROFILES_ARGS $SBT_MAVEN_TEST_ARGS"
+ echo "[info] Running Spark tests with these arguments: $SBT_MAVEN_PROFILES_ARGS ${SBT_MAVEN_TEST_ARGS[@]}"
# NOTE: 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
@@ -159,7 +162,7 @@ echo "========================================================================="
# QUESTION: Why doesn't 'yes "q"' work?
# QUESTION: Why doesn't 'grep -v -e "^\[info\] Resolving"' work?
echo -e "q\n" \
- | sbt/sbt "$SBT_MAVEN_PROFILES_ARGS" "$SBT_MAVEN_TEST_ARGS" \
+ | sbt/sbt "$SBT_MAVEN_PROFILES_ARGS" "${SBT_MAVEN_TEST_ARGS[@]}" \
| grep -v -e "info.*Resolving" -e "warn.*Merging" -e "info.*Including"
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org