Repository: spark Updated Branches: refs/heads/master 27fc536d9 -> 10e37f6eb
[UI][STREAMING] Modify the title, 'Records' instead of 'Input Size' ## What changes were proposed in this pull request? Spark Streaming is processing data should be record, so the title should be 'Records', and should not be 'Input Size'. Fix before: ![1](https://user-images.githubusercontent.com/26266482/30099599-c64d4a8a-9318-11e7-8a8d-1ca99b409323.png) Fix after: ![2](https://user-images.githubusercontent.com/26266482/30099609-cd4df7d0-9318-11e7-8a27-dbaec6797bb1.png) ## How was this patch tested? manual tests Please review http://spark.apache.org/contributing.html before opening a pull request. Author: guoxiaolong <guo.xiaolo...@zte.com.cn> Closes #19144 from guoxiaolongzte/streamingUI. Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/10e37f6e Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/10e37f6e Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/10e37f6e Branch: refs/heads/master Commit: 10e37f6eb6819c9233830c0d97e8fd1c713be0f1 Parents: 27fc536 Author: guoxiaolong <guo.xiaolo...@zte.com.cn> Authored: Fri Sep 22 11:51:57 2017 +0100 Committer: Sean Owen <so...@cloudera.com> Committed: Fri Sep 22 11:51:57 2017 +0100 ---------------------------------------------------------------------- .../scala/org/apache/spark/streaming/ui/AllBatchesTable.scala | 2 +- .../test/scala/org/apache/spark/streaming/UISeleniumSuite.scala | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/spark/blob/10e37f6e/streaming/src/main/scala/org/apache/spark/streaming/ui/AllBatchesTable.scala ---------------------------------------------------------------------- diff --git a/streaming/src/main/scala/org/apache/spark/streaming/ui/AllBatchesTable.scala b/streaming/src/main/scala/org/apache/spark/streaming/ui/AllBatchesTable.scala index 70b4bb4..f1070e9 100644 --- a/streaming/src/main/scala/org/apache/spark/streaming/ui/AllBatchesTable.scala +++ b/streaming/src/main/scala/org/apache/spark/streaming/ui/AllBatchesTable.scala @@ -25,7 +25,7 @@ private[ui] abstract class BatchTableBase(tableId: String, batchInterval: Long) protected def columns: Seq[Node] = { <th>Batch Time</th> - <th>Input Size</th> + <th>Records</th> <th>Scheduling Delay {SparkUIUtils.tooltip("Time taken by Streaming scheduler to submit jobs of a batch", "top")} </th> http://git-wip-us.apache.org/repos/asf/spark/blob/10e37f6e/streaming/src/test/scala/org/apache/spark/streaming/UISeleniumSuite.scala ---------------------------------------------------------------------- diff --git a/streaming/src/test/scala/org/apache/spark/streaming/UISeleniumSuite.scala b/streaming/src/test/scala/org/apache/spark/streaming/UISeleniumSuite.scala index e7cec99..f2204a1 100644 --- a/streaming/src/test/scala/org/apache/spark/streaming/UISeleniumSuite.scala +++ b/streaming/src/test/scala/org/apache/spark/streaming/UISeleniumSuite.scala @@ -121,11 +121,11 @@ class UISeleniumSuite h4Text.exists(_.matches("Completed Batches \\(last \\d+ out of \\d+\\)")) should be (true) findAll(cssSelector("""#active-batches-table th""")).map(_.text).toSeq should be { - List("Batch Time", "Input Size", "Scheduling Delay (?)", "Processing Time (?)", + List("Batch Time", "Records", "Scheduling Delay (?)", "Processing Time (?)", "Output Ops: Succeeded/Total", "Status") } findAll(cssSelector("""#completed-batches-table th""")).map(_.text).toSeq should be { - List("Batch Time", "Input Size", "Scheduling Delay (?)", "Processing Time (?)", + List("Batch Time", "Records", "Scheduling Delay (?)", "Processing Time (?)", "Total Delay (?)", "Output Ops: Succeeded/Total") } --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org For additional commands, e-mail: commits-h...@spark.apache.org