Github user hsaputra commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-40260830
Cool! Thanks @tgravescs
---
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 thi
Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-40216406
I pulled this into branch-1.0.
---
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 h
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/344
---
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 enabl
Github user mridulm commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-40178755
Merging this into master.
@tgravescs, @pwendell if this is relevant to 1.0, please do merge into that
branch too.
---
If your project is set up for it, you can reply
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39921610
All automated tests passed.
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13916/
---
If your project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39921609
Merged build finished. All automated tests 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39919863
Merged build triggered.
---
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 ha
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39919871
Merged build started.
---
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
Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39919770
Jenkins, test this please
---
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 t
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39894714
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13901/
---
If your project is set up for it, you can r
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39894712
Merged build finished.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39888097
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39888084
Merged build triggered.
---
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 ha
Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39885534
Oh its because jenkins is picking up an ip instead of when I run it locally
it picks up the host. Ok, let me fix that test.
---
If your project is set up for it, you c
Github user aarondav commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39885229
It seems that URI requires a scheme in the parameter, so `new
URI(appUIHostPort)` fails that check (which is part of the suite currently in
github).
---
If your project
Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39884822
Is jenkins not picking up the latest changes?
The test it says is failing works fine for me.
---
If your project is set up for it, you can reply to this email
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39882021
Merged build finished.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39882023
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13895/
---
If your project is set up for it, you can r
Github user hsaputra commented on a diff in the pull request:
https://github.com/apache/spark/pull/344#discussion_r11401846
--- Diff: core/src/main/scala/org/apache/spark/ui/SparkUI.scala ---
@@ -112,7 +112,12 @@ private[spark] class SparkUI(
logInfo("Stopped Spark Web UI a
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39877635
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39877613
Merged build triggered.
---
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 ha
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/344#discussion_r11401257
--- Diff: core/src/main/scala/org/apache/spark/ui/SparkUI.scala ---
@@ -112,7 +112,12 @@ private[spark] class SparkUI(
logInfo("Stopped Spark Web UI
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/344#discussion_r11397718
--- Diff: core/src/main/scala/org/apache/spark/ui/SparkUI.scala ---
@@ -112,7 +112,12 @@ private[spark] class SparkUI(
logInfo("Stopped Spark Web UI
Github user hsaputra commented on a diff in the pull request:
https://github.com/apache/spark/pull/344#discussion_r11397585
--- Diff: core/src/main/scala/org/apache/spark/ui/SparkUI.scala ---
@@ -112,7 +112,12 @@ private[spark] class SparkUI(
logInfo("Stopped Spark Web UI a
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39855171
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13892/
---
If your project is set up for it, you can r
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39855170
Merged build finished.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39850477
Merged build triggered.
---
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 ha
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39850494
Merged build started.
---
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
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/344#discussion_r11390149
--- Diff: core/src/main/scala/org/apache/spark/ui/SparkUI.scala ---
@@ -112,6 +112,11 @@ private[spark] class SparkUI(
logInfo("Stopped Spark Web UI
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39790128
Merged build finished.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39790131
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13857/
---
If your project is set up for it, you can r
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39786990
Merged build finished.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39786991
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13859/
---
If your project is set up for it, you can r
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/344#discussion_r11367039
--- Diff: core/src/main/scala/org/apache/spark/ui/SparkUI.scala ---
@@ -112,6 +112,11 @@ private[spark] class SparkUI(
logInfo("Stopped Spark Web UI a
Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39786047
Thanks for the review, updated it to remove extra newline and simplify the
assert. (and I accidentally hit the close button instead of the comment
button).
---
If yo
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39785928
Merged build started.
---
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
Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39785943
Thanks,
---
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
enabl
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39785918
Merged build triggered.
---
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 ha
GitHub user tgravescs reopened a pull request:
https://github.com/apache/spark/pull/344
SPARK-1417: Spark on Yarn - spark UI link from resourcemanager is broken
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tgravescs/spark SPAR
Github user tgravescs closed the pull request at:
https://github.com/apache/spark/pull/344
---
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 en
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/344#discussion_r11366365
--- Diff: core/src/main/scala/org/apache/spark/ui/SparkUI.scala ---
@@ -112,8 +112,14 @@ private[spark] class SparkUI(
logInfo("Stopped Spark Web UI
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/344#discussion_r11366350
--- Diff: core/src/test/scala/org/apache/spark/SparkUISuite.scala ---
@@ -0,0 +1,36 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39785420
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39785401
Merged build triggered.
---
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 ha
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/344#discussion_r11363051
--- Diff: core/src/main/scala/org/apache/spark/ui/SparkUI.scala ---
@@ -112,7 +112,10 @@ private[spark] class SparkUI(
logInfo("Stopped Spark Web UI
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/344#discussion_r11361448
--- Diff: core/src/main/scala/org/apache/spark/ui/SparkUI.scala ---
@@ -112,7 +112,10 @@ private[spark] class SparkUI(
logInfo("Stopped Spark Web UI a
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39773451
Merged build finished.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39773455
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13851/
---
If your project is set up for it, you can r
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39769248
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39769232
Merged build triggered.
---
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 ha
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39738667
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13843/
---
If your project is set up for it, you can r
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39738666
Merged build finished.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39733811
Merged build triggered.
---
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 ha
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/344#issuecomment-39733823
Merged build started.
---
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
GitHub user tgravescs opened a pull request:
https://github.com/apache/spark/pull/344
SPARK-1417: Spark on Yarn - spark UI link from resourcemanager is broken
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tgravescs/spark SPARK-
55 matches
Mail list logo