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

[guoxiaolongzte]

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:

How was this patch tested?

manual tests

Please review http://spark.apache.org/contributing.html before opening a pull request.

[AmplabJenkins]

Can one of the admins verify this patch?

[guoxiaolongzte]

@srowen @ajbozarth
Help to review the code, thanks.

[ajbozarth]

If this would be useful I don't see any harm in adding it, its a pretty small change

[srowen]

Hm, but do we do this consistently in the UI? show "Thing (count)"? it needs to be uniform if it's changed.

[guoxiaolongzte]

@srowen
As far as I know, we spark ui have two styles.

The first style is to display statistics separately, as follows:

The second style is displayed in the table header, with parentheses, as follows:

I am modifying this issue by using the second style.

[srowen]

I see. I think the first style exists to provide links that jump to running apps because it may be well below the fold if there are many workers. I think that can stay, even though the design of this little header is not great and wastes a lot of white space at the right.

I think it'd be nice to add (count) to all the tables of things in the UI, as I assume that's pretty easy to do, isn't obtrusive, and would make it more consistent. I suppose that goes for stages, tasks, etc. What do you think?

[guoxiaolongzte]

Yes, I agree with you. I will work on it.

[ajbozarth]

I like it, I think limiting this particular pr to updating the master ui is probably best for now, could you also update WorkerPage.scala as well since its part of the "Master UI"

[guoxiaolongzte]

@ajbozarth @srowen
At present my branch code ui about 'count ' to table , test results:
1.master page by i fix, as follows:

2.job page , as follows:

3.stage page , as follows:

4.task page, as follows:

5.work page, as follows:

[ajbozarth]

Thanks for checking @guoxiaolongzte looks like with this fix every table will now have a (count)

LGTM

[srowen]

Merged to master