[SPARK-7676] Bug fix and cleanup of stage timeline view

[kayousterhout]

cc @pwendell @sarutak

This commit cleans up some unnecessary code, eliminates the feature where when you mouse-over a box in the timeline, the corresponding task is highlighted in the table (because that feature is only useful in the rare case when you have a very small number of tasks, in which case it's easy to figure out the mapping anyway), and fixes a bug where nothing shows up if you try to visualize a stage with only 1 task.

[AmplabJenkins]

Merged build triggered.

[AmplabJenkins]

Merged build started.

[SparkQA]

Test build #32857 has started for PR 6202 at commit dfd29d4.

[sarutak]

Thanks for the bug fix and cleanup @kayousterhout !
I inspected the change and checked the appearance. In terms of removing the highlighting, I agree with you and how about adding a feature in order to jump to the corresponding row instead?
@pwendell , what do you think about the removal because you suggested this highlighting feature.

Once the highlighting thing is resolved, LGTM.

[pwendell]

I think @kayousterhout is right - we can remove the feature since it's only
useful in a small number of cases.

On Fri, May 15, 2015 at 3:54 PM, Kousuke Saruta notifications@github.com
wrote:

Thanks for the bug fix and cleanup @kayousterhout
https://github.com/kayousterhout !
I inspected the change and checked the appearance. In terms of removing
the highlighting, I agree with you and how about adding a feature in order
to jump to the corresponding row instead?
@pwendell https://github.com/pwendell , what do you think about the
removal because you suggested this highlighting feature.

Once the highlighting thing is resolved, LGTM.

—
Reply to this email directly or view it on GitHub
#6202 (comment).

[SparkQA]

Test build #32857 has finished for PR 6202 at commit dfd29d4.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Merged build finished. Test PASSed.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32857/
Test PASSed.

[kayousterhout]

@sarutak Let's do that as a separate PR (the feature so you can click on a link to jump to the correct row), just so we can get this cleanup in!