Skip to content

Commit 2d14ab7

Browse files
markgroverMarcelo Vanzin
authored andcommitted
[DOCS] Update docs to not suggest to package Spark before running tests.
## What changes were proposed in this pull request? Update docs to not suggest to package Spark before running tests. ## How was this patch tested? Not creating a JIRA since this pretty small. We haven't had the need to run mvn package before mvn test since 1.6 at least, or so I am told. So, updating the docs to not be misguiding. Author: Mark Grover <[email protected]> Closes #15572 from markgrover/doc_update.
1 parent 7f9ec19 commit 2d14ab7

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

docs/building-spark.md

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -217,9 +217,8 @@ For help in setting up IntelliJ IDEA or Eclipse for Spark development, and troub
217217
Tests are run by default via the [ScalaTest Maven plugin](http://www.scalatest.org/user_guide/using_the_scalatest_maven_plugin).
218218
Note that tests should not be run as root or an admin user.
219219

220-
Some of the tests require Spark to be packaged first, so always run `mvn package` with `-DskipTests` the first time. The following is an example of a correct (build, test) sequence:
220+
The following is an example of a command to run the tests:
221221

222-
./build/mvn -Pyarn -Phadoop-2.3 -DskipTests -Phive -Phive-thriftserver clean package
223222
./build/mvn -Pyarn -Phadoop-2.3 -Phive -Phive-thriftserver test
224223

225224
The ScalaTest plugin also supports running only a specific Scala test suite as follows:
@@ -233,9 +232,8 @@ or a Java test:
233232

234233
## Testing with SBT
235234

236-
Some of the tests require Spark to be packaged first, so always run `build/sbt package` the first time. The following is an example of a correct (build, test) sequence:
235+
The following is an example of a command to run the tests:
237236

238-
./build/sbt -Pyarn -Phadoop-2.3 -Phive -Phive-thriftserver package
239237
./build/sbt -Pyarn -Phadoop-2.3 -Phive -Phive-thriftserver test
240238

241239
To run only a specific test suite as follows:

0 commit comments

Comments
 (0)