Repository: spark
Updated Branches:
  refs/heads/master 99d579992 -> 8da6e8b1f


[SPARK-20720][WEB-UI] Executor Summary' should show the exact number, 'Removed 
Executors' should display the specific number, in the Application Page

## What changes were proposed in this pull request?

When the number of spark worker executors is large, if the specific number is 
displayed, will better help us to analyze and observe by spark ui.

Although this is a small improvement, but it is indeed very valuable.

After fix:
![executor1](https://cloud.githubusercontent.com/assets/26266482/25986597/2d8e4386-3723-11e7-9c24-e5bff17c26e2.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>
Author: 郭小龙 10207633 <guo.xiaolo...@zte.com.cn>
Author: guoxiaolongzte <guo.xiaolo...@zte.com.cn>

Closes #17961 from guoxiaolongzte/SPARK-20720.


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

Branch: refs/heads/master
Commit: 8da6e8b1f3e8f40cbacd323904d73fa0cb962689
Parents: 99d5799
Author: guoxiaolong <guo.xiaolo...@zte.com.cn>
Authored: Mon May 15 07:53:57 2017 +0100
Committer: Sean Owen <so...@cloudera.com>
Committed: Mon May 15 07:53:57 2017 +0100

----------------------------------------------------------------------
 .../apache/spark/deploy/master/ui/ApplicationPage.scala   |  4 ++--
 .../org/apache/spark/deploy/master/ui/MasterPage.scala    | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/8da6e8b1/core/src/main/scala/org/apache/spark/deploy/master/ui/ApplicationPage.scala
----------------------------------------------------------------------
diff --git 
a/core/src/main/scala/org/apache/spark/deploy/master/ui/ApplicationPage.scala 
b/core/src/main/scala/org/apache/spark/deploy/master/ui/ApplicationPage.scala
index 94ff81c..f408964 100644
--- 
a/core/src/main/scala/org/apache/spark/deploy/master/ui/ApplicationPage.scala
+++ 
b/core/src/main/scala/org/apache/spark/deploy/master/ui/ApplicationPage.scala
@@ -100,11 +100,11 @@ private[ui] class ApplicationPage(parent: MasterWebUI) 
extends WebUIPage("app")
 
       <div class="row-fluid"> <!-- Executors -->
         <div class="span12">
-          <h4> Executor Summary </h4>
+          <h4> Executor Summary ({allExecutors.length}) </h4>
           {executorsTable}
           {
             if (removedExecutors.nonEmpty) {
-              <h4> Removed Executors </h4> ++
+              <h4> Removed Executors ({removedExecutors.length}) </h4> ++
               removedExecutorsTable
             }
           }

http://git-wip-us.apache.org/repos/asf/spark/blob/8da6e8b1/core/src/main/scala/org/apache/spark/deploy/master/ui/MasterPage.scala
----------------------------------------------------------------------
diff --git 
a/core/src/main/scala/org/apache/spark/deploy/master/ui/MasterPage.scala 
b/core/src/main/scala/org/apache/spark/deploy/master/ui/MasterPage.scala
index ce71300..bc0bf6a 100644
--- a/core/src/main/scala/org/apache/spark/deploy/master/ui/MasterPage.scala
+++ b/core/src/main/scala/org/apache/spark/deploy/master/ui/MasterPage.scala
@@ -128,14 +128,14 @@ private[ui] class MasterPage(parent: MasterWebUI) extends 
WebUIPage("") {
 
         <div class="row-fluid">
           <div class="span12">
-            <h4> Workers </h4>
+            <h4> Workers ({workers.length}) </h4>
             {workerTable}
           </div>
         </div>
 
         <div class="row-fluid">
           <div class="span12">
-            <h4 id="running-app"> Running Applications </h4>
+            <h4 id="running-app"> Running Applications ({activeApps.length}) 
</h4>
             {activeAppsTable}
           </div>
         </div>
@@ -144,7 +144,7 @@ private[ui] class MasterPage(parent: MasterWebUI) extends 
WebUIPage("") {
           {if (hasDrivers) {
              <div class="row-fluid">
                <div class="span12">
-                 <h4> Running Drivers </h4>
+                 <h4> Running Drivers ({activeDrivers.length}) </h4>
                  {activeDriversTable}
                </div>
              </div>
@@ -154,7 +154,7 @@ private[ui] class MasterPage(parent: MasterWebUI) extends 
WebUIPage("") {
 
         <div class="row-fluid">
           <div class="span12">
-            <h4 id="completed-app"> Completed Applications </h4>
+            <h4 id="completed-app"> Completed Applications 
({completedApps.length}) </h4>
             {completedAppsTable}
           </div>
         </div>
@@ -164,7 +164,7 @@ private[ui] class MasterPage(parent: MasterWebUI) extends 
WebUIPage("") {
             if (hasDrivers) {
               <div class="row-fluid">
                 <div class="span12">
-                  <h4> Completed Drivers </h4>
+                  <h4> Completed Drivers ({completedDrivers.length}) </h4>
                   {completedDriversTable}
                 </div>
               </div>


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to