You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by va...@apache.org on 2016/09/29 23:01:52 UTC

spark git commit: [SPARK-17412][DOC] All test should not be run by `root` or any admin user

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 <do...@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 <do...@apache.org>
Authored: Thu Sep 29 16:01:45 2016 -0700
Committer: Marcelo Vanzin <va...@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-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org