From 39eb3bb1ec29aa993de13a6eba3ab27db6fc5371 Mon Sep 17 00:00:00 2001
From: Dongjoon Hyun <dongjoon@apache.org>
Date: Thu, 29 Sep 2016 16:01:45 -0700
Subject: [PATCH] [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 <dongjoon@apache.org>

Closes #15291 from dongjoon-hyun/SPARK-17412.
---
 docs/building-spark.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/docs/building-spark.md b/docs/building-spark.md
index 75c304a3cc..da7eeb8348 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:
 
-- 
GitLab