You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by jo...@apache.org on 2015/01/12 20:01:24 UTC

spark git commit: SPARK-4159 [BUILD] Addendum: improve running of single test after enabling Java tests

Repository: spark
Updated Branches:
  refs/heads/master ef9224e08 -> 13e610b88


SPARK-4159 [BUILD] Addendum: improve running of single test after enabling Java tests

https://issues.apache.org/jira/browse/SPARK-4159 was resolved but as Sandy points out, the guidance in https://cwiki.apache.org/confluence/display/SPARK/Useful+Developer+Tools under "Running Individual Tests" no longer quite works, not optimally.

This minor change is not really the important change, which is an update to the wiki text. The correct way to run one Scala test suite in Maven is now:

```
mvn test -DwildcardSuites=org.apache.spark.io.CompressionCodecSuite -Dtests=none
```

The correct way to run one Java test is

```
mvn test -DwildcardSuites=none -Dtests=org.apache.spark.streaming.JavaAPISuite
```

Basically, you have to set two properties in order to suppress all of one type of test (with a non-existent test name like 'none') and all but one test of the other type.

The change in the PR just prevents Surefire from barfing when it finds no "none" test.

Author: Sean Owen <so...@cloudera.com>

Closes #3993 from srowen/SPARK-4159 and squashes the following commits:

83106d7 [Sean Owen] Default failIfNoTests to false to enable the -DwildcardSuites=... -Dtests=... syntax for running one test to work


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

Branch: refs/heads/master
Commit: 13e610b88eb83b11785737ab1d99927f676f81c6
Parents: ef9224e
Author: Sean Owen <so...@cloudera.com>
Authored: Mon Jan 12 11:00:56 2015 -0800
Committer: Josh Rosen <jo...@databricks.com>
Committed: Mon Jan 12 11:01:00 2015 -0800

----------------------------------------------------------------------
 pom.xml | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/13e610b8/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index aadcdfd..0403c00 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1130,6 +1130,7 @@
               <spark.executor.extraClassPath>${test_classpath}</spark.executor.extraClassPath>
               <spark.driver.allowMultipleContexts>true</spark.driver.allowMultipleContexts>
             </systemProperties>
+            <failIfNoTests>false</failIfNoTests>
           </configuration>
         </plugin>
         <!-- Scalatest runs all Scala tests -->


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