[GitHub] spark pull request: Don't include the empty string as a default...

2014-09-05 Thread ash211
GitHub user ash211 opened a pull request: https://github.com/apache/spark/pull/2286 Don't include the empty string as a defaultAclUser Changes logging from ``` 14/09/05 02:01:08 INFO SecurityManager: Changing view acls to: aash, 14/09/05 02:01:08 INFO

[GitHub] spark pull request: Don't include the empty string as a default...

2014-09-05 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2286#issuecomment-54604796 [QA tests have started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19804/consoleFull) for PR 2286 at commit

[GitHub] spark pull request: Don't include the empty string as a default...

2014-09-05 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2286#issuecomment-54610134 [QA tests have finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/19804/consoleFull) for PR 2286 at commit

[GitHub] spark pull request: Don't include the empty string as a default...

2014-09-05 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/2286#discussion_r17174246 --- Diff: core/src/main/scala/org/apache/spark/SecurityManager.scala --- @@ -162,7 +162,7 @@ private[spark] class SecurityManager(sparkConf: SparkConf)

[GitHub] spark pull request: Don't include the empty string as a default...

2014-09-05 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/2286#issuecomment-54630202 Thanks for working on this, I've been meaning to fix this for a while. Could you also please file a jira and link them. The header of the pr should include