[GitHub] spark pull request: [SPARK-5078] Optionally read from SPARK_LOCAL_...

2015-01-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/3893 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark pull request: [SPARK-5078] Optionally read from SPARK_LOCAL_...

2015-01-12 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/3893#issuecomment-69634618 Okay let's pull this in for now. Don't want to block this on `SPARK-5113`. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-5078] Optionally read from SPARK_LOCAL_...

2015-01-06 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/3893#discussion_r22512543 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -701,7 +701,7 @@ private[spark] object Utils extends Logging { } }

[GitHub] spark pull request: [SPARK-5078] Optionally read from SPARK_LOCAL_...

2015-01-06 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/3893#discussion_r22512468 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -701,7 +701,7 @@ private[spark] object Utils extends Logging { } }

[GitHub] spark pull request: [SPARK-5078] Optionally read from SPARK_LOCAL_...

2015-01-06 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/3893#issuecomment-68934185 I put some thoughts on a new JIRA about how to clean this up overall in Spark - as for this patch. I'm fine to merge it, but it would be good if we did a proper

[GitHub] spark pull request: [SPARK-5078] Optionally read from SPARK_LOCAL_...

2015-01-06 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/3893#discussion_r22538331 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -701,7 +701,7 @@ private[spark] object Utils extends Logging { } }

[GitHub] spark pull request: [SPARK-5078] Optionally read from SPARK_LOCAL_...

2015-01-04 Thread marmbrus
GitHub user marmbrus opened a pull request: https://github.com/apache/spark/pull/3893 [SPARK-5078] Optionally read from SPARK_LOCAL_HOSTNAME Current spark lets you set the ip address using SPARK_LOCAL_IP, but then this is given to akka after doing a reverse DNS lookup. This makes

[GitHub] spark pull request: [SPARK-5078] Optionally read from SPARK_LOCAL_...

2015-01-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3893#issuecomment-68658688 [Test build #25039 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25039/consoleFull) for PR 3893 at commit

[GitHub] spark pull request: [SPARK-5078] Optionally read from SPARK_LOCAL_...

2015-01-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3893#issuecomment-68661673 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5078] Optionally read from SPARK_LOCAL_...

2015-01-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3893#issuecomment-68661671 [Test build #25039 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25039/consoleFull) for PR 3893 at commit