Repository: spark Updated Branches: refs/heads/master 3993ebca2 -> 39eb3bb1e
[SPARK-17412][DOC] All test should not be run by `root` or any admin user ## What changes were proposed in this pull request? `FsHistoryProviderSuite` fails if `root` user runs it. The test case **SPARK-3697: ignore directories that cannot be read** depends on `setReadable(false, false)` to make test data files and expects the number of accessible files is 1. But, `root` can access all files, so it returns 2. This PR adds the assumption explicitly on doc. `building-spark.md`. ## How was this patch tested? This is a documentation change. Author: Dongjoon Hyun <dongj...@apache.org> Closes #15291 from dongjoon-hyun/SPARK-17412. Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/39eb3bb1 Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/39eb3bb1 Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/39eb3bb1 Branch: refs/heads/master Commit: 39eb3bb1ec29aa993de13a6eba3ab27db6fc5371 Parents: 3993ebc Author: Dongjoon Hyun <dongj...@apache.org> Authored: Thu Sep 29 16:01:45 2016 -0700 Committer: Marcelo Vanzin <van...@cloudera.com> Committed: Thu Sep 29 16:01:45 2016 -0700 ---------------------------------------------------------------------- docs/building-spark.md | 1 + 1 file changed, 1 insertion(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/spark/blob/39eb3bb1/docs/building-spark.md ---------------------------------------------------------------------- diff --git a/docs/building-spark.md b/docs/building-spark.md index 75c304a3..da7eeb8 100644 --- a/docs/building-spark.md +++ b/docs/building-spark.md @@ -215,6 +215,7 @@ For help in setting up IntelliJ IDEA or Eclipse for Spark development, and troub # Running Tests Tests are run by default via the [ScalaTest Maven plugin](http://www.scalatest.org/user_guide/using_the_scalatest_maven_plugin). +Note that tests should not be run as root or an admin user. 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: --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org For additional commands, e-mail: commits-h...@spark.apache.org
