diff options
author | Masayoshi TSUZUKI <tsudukim@oss.nttdata.co.jp> | 2014-07-10 01:18:37 -0700 |
---|---|---|
committer | Reynold Xin <rxin@apache.org> | 2014-07-10 01:18:37 -0700 |
commit | c2babc089bd97137d240ca119ec4fc4b5a1422c8 (patch) | |
tree | 2b0b6c4eb1433b348d8cd6855910b7c0d6156d9d /core/src/main/scala/org | |
parent | 2b18ea9826395177ac2203dbf8eb37c220ab8e67 (diff) | |
download | spark-c2babc089bd97137d240ca119ec4fc4b5a1422c8.tar.gz spark-c2babc089bd97137d240ca119ec4fc4b5a1422c8.tar.bz2 spark-c2babc089bd97137d240ca119ec4fc4b5a1422c8.zip |
SPARK-2115: Stage kill link is too close to stage details link
Moved (kill) link to the right side. Add confirmation dialog when (kill) link is clicked.
Author: Masayoshi TSUZUKI <tsudukim@oss.nttdata.co.jp>
Closes #1350 from tsudukim/feature/SPARK-2115 and squashes the following commits:
e2263b0 [Masayoshi TSUZUKI] Moved (kill) link to the right side. Add confirmation dialog when (kill) link is clicked.
Diffstat (limited to 'core/src/main/scala/org')
-rw-r--r-- | core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala b/core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala index 4013c6f499..fd8d0b5cdd 100644 --- a/core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala +++ b/core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala @@ -89,7 +89,8 @@ private[ui] class StageTableBase( // scalastyle:off val killLink = if (killEnabled) { <span class="kill-link"> - (<a href={"%s/stages/stage/kill?id=%s&terminate=true".format(UIUtils.prependBaseUri(basePath), s.stageId)}>kill</a>) + (<a href={"%s/stages/stage/kill?id=%s&terminate=true".format(UIUtils.prependBaseUri(basePath), s.stageId)} + onclick={"return window.confirm('Are you sure you want to kill stage %s ?');".format(s.stageId)}>kill</a>) </span> } // scalastyle:on @@ -109,7 +110,7 @@ private[ui] class StageTableBase( listener.stageIdToDescription.get(s.stageId) .map(d => <div><em>{d}</em></div><div>{nameLink} {killLink}</div>) - .getOrElse(<div>{killLink} {nameLink} {details}</div>) + .getOrElse(<div>{nameLink} {killLink} {details}</div>) } protected def stageRow(s: StageInfo): Seq[Node] = { |