Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11644
---
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 ena
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195928608
Merged to master and 1.6. I added 1.6 since it's just a tiny doc
improvement, and probably adds disproportionate value.
---
If your project is set up for it, you can re
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195655516
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195655515
Merged build finished. Test PASSed.
---
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195655145
**[Test build #52980 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52980/consoleFull)**
for PR 11644 at commit
[`3a0e3f1`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195650306
Merged build finished. Test PASSed.
---
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195650307
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195650234
**[Test build #52976 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52976/consoleFull)**
for PR 11644 at commit
[`0d8303e`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195628886
**[Test build #52980 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52980/consoleFull)**
for PR 11644 at commit
[`3a0e3f1`](https://gi
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11644#discussion_r55911638
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2014,8 +2014,10 @@ private[spark] object Utils extends Logging {
} catch {
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195622365
**[Test build #52976 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52976/consoleFull)**
for PR 11644 at commit
[`0d8303e`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195620583
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195620580
**[Test build #52975 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52975/consoleFull)**
for PR 11644 at commit
[`933e205`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195620582
Merged build finished. Test FAILed.
---
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195620424
**[Test build #52975 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52975/consoleFull)**
for PR 11644 at commit
[`933e205`](https://gi
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11644#discussion_r55908574
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2014,8 +2014,10 @@ private[spark] object Utils extends Logging {
} catch {
Github user bjornjon commented on a diff in the pull request:
https://github.com/apache/spark/pull/11644#discussion_r55908378
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2014,8 +2014,10 @@ private[spark] object Utils extends Logging {
} catch {
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195604878
looks good
---
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
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/11644#discussion_r55905764
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2014,8 +2014,10 @@ private[spark] object Utils extends Logging {
} catch
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195604553
Merged build finished. Test PASSed.
---
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195604555
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195604349
**[Test build #52948 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52948/consoleFull)**
for PR 11644 at commit
[`8177e69`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195551694
**[Test build #52948 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52948/consoleFull)**
for PR 11644 at commit
[`8177e69`](https://gi
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195549744
ok to test
---
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
ena
Github user squito commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195451612
bjorn and I chatted about this a bit more offline, I understand why its a
bit complicated for the non-UI port a bit more, but also feel this simple
change is good enough
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/11644#discussion_r55849588
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2014,8 +2014,10 @@ private[spark] object Utils extends Logging {
} catch {
Github user squito commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195429708
I can't see the earlier version of this diff, but I dont' understand why
you'd need to introduce a `SparkEnv`. I see that currently, when a `WebUI` is
created, you no l
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195342004
LGTM. (By the way Bjorn fields a lot of support issues for Spark and this
is apparently a frequent source of questions/confusion. Great if a simple
message change can pr
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195248388
I like the principle, but passing around an error message as an arg to
`SparkEnv` makes this tiny aspect leak too much into the API. If it's just to
be able to report wh
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11644#issuecomment-195166923
Can one of the admins verify this patch?
---
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 p
GitHub user bjornjon opened a pull request:
https://github.com/apache/spark/pull/11644
[SPARK-13810] [CORE] Add Port Configuration Suggestions on Bind Exceptions
## What changes were proposed in this pull request?
Currently, when a java.net.BindException is thrown, it displays th
31 matches
Mail list logo