[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-112556515 I think this would have to be rebased at least, and rewritten to match the new kill link behavior elsewhere. Can this be closed? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user lianhuiwang closed the pull request at: https://github.com/apache/spark/pull/4823 --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-85783347 [Test build #29135 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29135/consoleFull) for PR 4823 at commit [`a0eee0c`](https://github.com/apache/spark/commit/a0eee0caa14824cefb99d178522f6ada2a305f4a). * This patch **does not merge cleanly**. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-85809039 [Test build #29135 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29135/consoleFull) for PR 4823 at commit [`a0eee0c`](https://github.com/apache/spark/commit/a0eee0caa14824cefb99d178522f6ada2a305f4a). * This patch **fails Spark unit tests**. * This patch **does not merge cleanly**. * This patch adds no public classes. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-85809092 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29135/ 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 project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user lianhuiwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/4823#discussion_r27089597
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala ---
@@ -42,6 +42,18 @@ private[ui] class AllJobsPage(parent: JobsTab) extends
WebUIPage() {
}
def makeRow(job: JobUIData): Seq[Node] = {
+ // scalastyle:off
+ val killLink = if (killEnabled) {
+val killLinkUri =
s${UIUtils.prependBaseUri(parent.basePath)}/jobs/job/kill? +
+ sid=${job.jobId}terminate=true
+val confirm = return window.confirm( +
+ s'Are you sure you want to kill job ${job.jobId} ?');
+span class=kill-link
+ (a href={killLinkUri} onclick={confirm}kill/a)
+/span
+ }
+ // scalastyle:on
--- End diff --
yes, if killEnabled = false, the type of killLink is Any.
i will add an else case that returns Seq.empty.
---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-85415514 [Test build #29064 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29064/consoleFull) for PR 4823 at commit [`ba16839`](https://github.com/apache/spark/commit/ba168399f4ee4f59a2c0568b9e094b55747e97c0). * This patch **passes all tests**. * This patch **does not merge cleanly**. * This patch adds no public classes. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-85415556 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29064/ 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 project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/4823#discussion_r27059177
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala ---
@@ -42,6 +42,18 @@ private[ui] class AllJobsPage(parent: JobsTab) extends
WebUIPage() {
}
def makeRow(job: JobUIData): Seq[Node] = {
+ // scalastyle:off
+ val killLink = if (killEnabled) {
+val killLinkUri =
s${UIUtils.prependBaseUri(parent.basePath)}/jobs/job/kill? +
+ sid=${job.jobId}terminate=true
+val confirm = return window.confirm( +
+ s'Are you sure you want to kill job ${job.jobId} ?');
+span class=kill-link
+ (a href={killLinkUri} onclick={confirm}kill/a)
+/span
+ }
+ // scalastyle:on
--- End diff --
also, I'm not sure what the type of `killLink` is. It seems to me to be
`Any` because there's no else case. Can you make it explicit by adding an else
case that returns `Seq.empty`?
---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user lianhuiwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/4823#discussion_r27006782
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala ---
@@ -42,6 +42,18 @@ private[ui] class AllJobsPage(parent: JobsTab) extends
WebUIPage() {
}
def makeRow(job: JobUIData): Seq[Node] = {
+ // scalastyle:off
+ val killLink = if (killEnabled) {
+val killLinkUri =
s${UIUtils.prependBaseUri(parent.basePath)}/jobs/job/kill? +
+ sid=${job.jobId}terminate=true
+val confirm = return window.confirm( +
+ s'Are you sure you want to kill job ${job.jobId} ?');
+span class=kill-link
+ (a href={killLinkUri} onclick={confirm}kill/a)
+/span
+ }
+ // scalastyle:on
--- End diff --
because here
https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala#L76
has scalastyle.
---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-85376189 [Test build #29064 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29064/consoleFull) for PR 4823 at commit [`ba16839`](https://github.com/apache/spark/commit/ba168399f4ee4f59a2c0568b9e094b55747e97c0). * This patch **does not merge cleanly**. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-85319185 @JoshRosen @kayousterhout what do you think --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/4823#discussion_r27000467
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/StagesTab.scala ---
@@ -34,12 +34,15 @@ private[ui] class StagesTab(parent: SparkUI) extends
SparkUITab(parent, stages
def isFairScheduler = listener.schedulingMode.exists(_ ==
SchedulingMode.FAIR)
- def handleKillRequest(request: HttpServletRequest) = {
-if ((killEnabled)
(parent.securityManager.checkModifyPermissions(request.getRemoteUser))) {
+ def handleKillRequest(request: HttpServletRequest) = {
--- End diff --
please add `: Unit` return type here
---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/4823#discussion_r27000488
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala ---
@@ -42,6 +42,18 @@ private[ui] class AllJobsPage(parent: JobsTab) extends
WebUIPage() {
}
def makeRow(job: JobUIData): Seq[Node] = {
+ // scalastyle:off
+ val killLink = if (killEnabled) {
+val killLinkUri =
s${UIUtils.prependBaseUri(parent.basePath)}/jobs/job/kill? +
+ sid=${job.jobId}terminate=true
+val confirm = return window.confirm( +
+ s'Are you sure you want to kill job ${job.jobId} ?');
+span class=kill-link
+ (a href={killLinkUri} onclick={confirm}kill/a)
+/span
+ }
+ // scalastyle:on
--- End diff --
do you need to disable scalastyle here?
---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76588669 OK by me; would be good to let @kayousterhout give it a thumbs up. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76587368 [Test build #28142 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28142/consoleFull) for PR 4823 at commit [`25fc0fd`](https://github.com/apache/spark/commit/25fc0fd1fc574522ab08f23f6f61673960a1072a). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76587372 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28142/ 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 project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/4823#discussion_r25558253
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/JobsTab.scala ---
@@ -29,4 +31,18 @@ private[ui] class JobsTab(parent: SparkUI) extends
SparkUITab(parent, jobs) {
attachPage(new AllJobsPage(this))
attachPage(new JobPage(this))
+
+ def handleKillRequest(request: HttpServletRequest) = {
--- End diff --
Nit: extra space before brace, and extra parens around `killEnabled`.
Also I slightly wonder about
`Option(request.getParameter(id)).getOrElse(-1).toInt` -- I doubt there
would ever be a negative job ID but seems avoidable by just checking whether
the param exists rather than filling in -1 for missing params. Still I don't
think this does any harm at the moment.
---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/4823#discussion_r25558245
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala ---
@@ -42,6 +42,18 @@ private[ui] class AllJobsPage(parent: JobsTab) extends
WebUIPage() {
}
def makeRow(job: JobUIData): Seq[Node] = {
+ // scalastyle:off
+ val killLink = if (killEnabled) {
+val killLinkUri = %s/jobs/job/kill?id=%sterminate=true
--- End diff --
Just a few tiny comments to consider, if you have to otherwise make changes
to this PR: this might be easier to read with string interpolation
---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76586464 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28140/ 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 project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76586461 [Test build #28140 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28140/consoleFull) for PR 4823 at commit [`584240a`](https://github.com/apache/spark/commit/584240affe2422e167b4d3ea87b5766623ed72f6). * This patch **fails Spark unit tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76582533 [Test build #28140 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28140/consoleFull) for PR 4823 at commit [`584240a`](https://github.com/apache/spark/commit/584240affe2422e167b4d3ea87b5766623ed72f6). * This patch merges cleanly. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user lianhuiwang commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76584418 @srowen i have update for your comments. can you take a look again. 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 enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76584498 [Test build #28142 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28142/consoleFull) for PR 4823 at commit [`25fc0fd`](https://github.com/apache/spark/commit/25fc0fd1fc574522ab08f23f6f61673960a1072a). * This patch merges cleanly. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76511755 [Test build #28112 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28112/consoleFull) for PR 4823 at commit [`7f52874`](https://github.com/apache/spark/commit/7f52874badfea314d019b0dc9097c54b8af2f654). * This patch merges cleanly. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76508165 [Test build #28107 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28107/consoleFull) for PR 4823 at commit [`af461cc`](https://github.com/apache/spark/commit/af461ccce44e2792ea9356ccc2db6c84609511a0). * This patch merges cleanly. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76510590 [Test build #28107 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28107/consoleFull) for PR 4823 at commit [`af461cc`](https://github.com/apache/spark/commit/af461ccce44e2792ea9356ccc2db6c84609511a0). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76510593 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28107/ 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 project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76514247 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28112/ 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 project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4823#issuecomment-76514243 [Test build #28112 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28112/consoleFull) for PR 4823 at commit [`7f52874`](https://github.com/apache/spark/commit/7f52874badfea314d019b0dc9097c54b8af2f654). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4411][UI]Add kill link for jobs in the ...
GitHub user lianhuiwang opened a pull request: https://github.com/apache/spark/pull/4823 [SPARK-4411][UI]Add kill link for jobs in the UI We should have a kill link for each job, similar to what we have for each stage, so it's easier for users to kill jobs in the UI. @kayousterhout can you take a look at this? thanks. You can merge this pull request into a Git repository by running: $ git pull https://github.com/lianhuiwang/spark SPARK-4411 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/4823.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #4823 commit af461ccce44e2792ea9356ccc2db6c84609511a0 Author: Lianhui Wang lianhuiwan...@gmail.com Date: 2015-02-28T03:24:46Z Add kill link for jobs in the UI --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
