Updated Branches:
  refs/heads/master dfd1ebc2d -> b8bf04a08

Fix bug JIRA-923


Project: http://git-wip-us.apache.org/repos/asf/incubator-spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-spark/commit/b3208063
Tree: http://git-wip-us.apache.org/repos/asf/incubator-spark/tree/b3208063
Diff: http://git-wip-us.apache.org/repos/asf/incubator-spark/diff/b3208063

Branch: refs/heads/master
Commit: b3208063afe7190efd6e54a41b7af28d15c46358
Parents: 714fdab
Author: Andrew xia <[email protected]>
Authored: Mon Nov 11 15:39:10 2013 +0800
Committer: Andrew xia <[email protected]>
Committed: Mon Nov 11 15:39:10 2013 +0800

----------------------------------------------------------------------
 .../apache/spark/ui/UIWorkloadGenerator.scala   |  2 +-
 .../org/apache/spark/ui/jobs/StagePage.scala    | 24 ++++++++++++++++----
 .../org/apache/spark/ui/jobs/StageTable.scala   | 11 +++++----
 3 files changed, 28 insertions(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-spark/blob/b3208063/core/src/main/scala/org/apache/spark/ui/UIWorkloadGenerator.scala
----------------------------------------------------------------------
diff --git a/core/src/main/scala/org/apache/spark/ui/UIWorkloadGenerator.scala 
b/core/src/main/scala/org/apache/spark/ui/UIWorkloadGenerator.scala
index 453394d..fcd1b51 100644
--- a/core/src/main/scala/org/apache/spark/ui/UIWorkloadGenerator.scala
+++ b/core/src/main/scala/org/apache/spark/ui/UIWorkloadGenerator.scala
@@ -35,7 +35,7 @@ private[spark] object UIWorkloadGenerator {
 
   def main(args: Array[String]) {
     if (args.length < 2) {
-      println("usage: ./spark-class spark.ui.UIWorkloadGenerator [master] 
[FIFO|FAIR]")
+      println("usage: ./spark-class org.apache.spark.ui.UIWorkloadGenerator 
[master] [FIFO|FAIR]")
       System.exit(1)
     }
     val master = args(0)

http://git-wip-us.apache.org/repos/asf/incubator-spark/blob/b3208063/core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala
----------------------------------------------------------------------
diff --git a/core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala 
b/core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala
index 163a374..9316465 100644
--- a/core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala
+++ b/core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala
@@ -152,6 +152,20 @@ private[spark] class StagePage(parent: JobProgressUI) {
       else metrics.map(m => 
parent.formatDuration(m.executorRunTime)).getOrElse("")
     val gcTime = metrics.map(m => m.jvmGCTime).getOrElse(0L)
 
+    var shuffleReadSortable: String = ""
+    var shuffleReadReadable: String = ""
+    if (shuffleRead) {
+      shuffleReadSortable = metrics.flatMap{m => m.shuffleReadMetrics}.map{s 
=> s.remoteBytesRead}.toString()
+      shuffleReadReadable = metrics.flatMap{m => m.shuffleReadMetrics}.map{s 
=>Utils.bytesToString(s.remoteBytesRead)}.getOrElse("")
+    }
+
+    var shuffleWriteSortable: String = ""
+    var shuffleWriteReadable: String = ""
+    if (shuffleWrite) {
+      shuffleWriteSortable = metrics.flatMap{m => m.shuffleWriteMetrics}.map{s 
=> s.shuffleBytesWritten}.toString()
+      shuffleWriteReadable = metrics.flatMap{m => m.shuffleWriteMetrics}.map{s 
=>Utils.bytesToString(s.shuffleBytesWritten)}.getOrElse("")
+    }
+
     <tr>
       <td>{info.taskId}</td>
       <td>{info.status}</td>
@@ -165,12 +179,14 @@ private[spark] class StagePage(parent: JobProgressUI) {
         {if (gcTime > 0) parent.formatDuration(gcTime) else ""}
       </td>
       {if (shuffleRead) {
-        <td>{metrics.flatMap{m => m.shuffleReadMetrics}.map{s =>
-          Utils.bytesToString(s.remoteBytesRead)}.getOrElse("")}</td>
+        <td sorttable_customkey={shuffleReadSortable}>
+          {shuffleReadReadable}
+        </td>
       }}
       {if (shuffleWrite) {
-        <td>{metrics.flatMap{m => m.shuffleWriteMetrics}.map{s =>
-          Utils.bytesToString(s.shuffleBytesWritten)}.getOrElse("")}</td>
+        <td sorttable_customkey={shuffleWriteSortable}>
+        {shuffleWriteReadable}
+        </td>
       }}
       <td>{exception.map(e =>
         <span>

http://git-wip-us.apache.org/repos/asf/incubator-spark/blob/b3208063/core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala
----------------------------------------------------------------------
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 07db862..c9ee66f 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
@@ -79,11 +79,14 @@ private[spark] class StageTable(val stages: Seq[Stage], val 
parent: JobProgressU
       case None => "Unknown"
     }
 
-    val shuffleRead = listener.stageToShuffleRead.getOrElse(s.id, 0L) match {
+    val shuffleReadSortable = listener.stageToShuffleRead.getOrElse(s.id, 0L)
+    val shuffleRead = shuffleReadSortable match {
       case 0 => ""
       case b => Utils.bytesToString(b)
     }
-    val shuffleWrite = listener.stageToShuffleWrite.getOrElse(s.id, 0L) match {
+
+    val shuffleWriteSortable = listener.stageToShuffleWrite.getOrElse(s.id, 0L)
+    val shuffleWrite = shuffleWriteSortable match {
       case 0 => ""
       case b => Utils.bytesToString(b)
     }
@@ -119,8 +122,8 @@ private[spark] class StageTable(val stages: Seq[Stage], val 
parent: JobProgressU
       <td class="progress-cell">
         {makeProgressBar(startedTasks, completedTasks, failedTasks, 
totalTasks)}
       </td>
-      <td>{shuffleRead}</td>
-      <td>{shuffleWrite}</td>
+      <td 
sorttable_customekey={shuffleReadSortable.toString}>{shuffleRead}</td>
+      <td 
sorttable_customekey={shuffleWriteSortable.toString}>{shuffleWrite}</td>
     </tr>
   }
 }

Reply via email to