You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by we...@apache.org on 2017/07/03 05:31:05 UTC
spark git commit: [SPARK-21250][WEB-UI] Add a url in the table of
'Running Executors' in worker page to visit job page.
Repository: spark
Updated Branches:
refs/heads/master d4107196d -> d913db16a
[SPARK-21250][WEB-UI] Add a url in the table of 'Running Executors' in worker page to visit job page.
## What changes were proposed in this pull request?
Add a url in the table of 'Running Executors' in worker page to visit job page.
When I click URL of 'Name', the current page jumps to the job page. Of course this is only in the table of 'Running Executors'.
This URL of 'Name' is in the table of 'Finished Executors' does not exist, the click will not jump to any page.
fix before:
fix after:
## How was this patch tested?
manual tests
Please review http://spark.apache.org/contributing.html before opening a pull request.
Author: guoxiaolong <gu...@zte.com.cn>
Closes #18464 from guoxiaolongzte/SPARK-21250.
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/d913db16
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/d913db16
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/d913db16
Branch: refs/heads/master
Commit: d913db16a0de0983961f9d0c5f9b146be7226ac1
Parents: d410719
Author: guoxiaolong <gu...@zte.com.cn>
Authored: Mon Jul 3 13:31:01 2017 +0800
Committer: Wenchen Fan <we...@databricks.com>
Committed: Mon Jul 3 13:31:01 2017 +0800
----------------------------------------------------------------------
.../org/apache/spark/deploy/worker/ui/WorkerPage.scala | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/spark/blob/d913db16/core/src/main/scala/org/apache/spark/deploy/worker/ui/WorkerPage.scala
----------------------------------------------------------------------
diff --git a/core/src/main/scala/org/apache/spark/deploy/worker/ui/WorkerPage.scala b/core/src/main/scala/org/apache/spark/deploy/worker/ui/WorkerPage.scala
index 1ad9731..ea39b0d 100644
--- a/core/src/main/scala/org/apache/spark/deploy/worker/ui/WorkerPage.scala
+++ b/core/src/main/scala/org/apache/spark/deploy/worker/ui/WorkerPage.scala
@@ -23,8 +23,8 @@ import scala.xml.Node
import org.json4s.JValue
+import org.apache.spark.deploy.{ExecutorState, JsonProtocol}
import org.apache.spark.deploy.DeployMessages.{RequestWorkerState, WorkerStateResponse}
-import org.apache.spark.deploy.JsonProtocol
import org.apache.spark.deploy.master.DriverState
import org.apache.spark.deploy.worker.{DriverRunner, ExecutorRunner}
import org.apache.spark.ui.{UIUtils, WebUIPage}
@@ -112,7 +112,15 @@ private[ui] class WorkerPage(parent: WorkerWebUI) extends WebUIPage("") {
<td>
<ul class="unstyled">
<li><strong>ID:</strong> {executor.appId}</li>
- <li><strong>Name:</strong> {executor.appDesc.name}</li>
+ <li><strong>Name:</strong>
+ {
+ if ({executor.state == ExecutorState.RUNNING} && executor.appDesc.appUiUrl.nonEmpty) {
+ <a href={executor.appDesc.appUiUrl}> {executor.appDesc.name}</a>
+ } else {
+ {executor.appDesc.name}
+ }
+ }
+ </li>
<li><strong>User:</strong> {executor.appDesc.user}</li>
</ul>
</td>
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org