Skip to content

Conversation

@pgandhi999
Copy link

Task Summary Metrics Table on Stage Page shows empty table when no data is present instead of showing a message.

What changes were proposed in this pull request?

Added a custom message to show on the task summary metrics table as well as executor summary table when no data is present.

How was this patch tested?

Before:

49335550-29277d00-f615-11e8-8e62-a953e76bcebf

After:

screen shot 2018-12-03 at 1 56 09 pm

…table when no data is present

Added a custom message to show on the task summary metrics table as well as executor summary table when no data is present.
@pgandhi999
Copy link
Author

cc @tgravescs @abellina

@shahidki31
Copy link
Contributor

Thanks @pgandhi999 👍

@SparkQA
Copy link

SparkQA commented Dec 3, 2018

Test build #99600 has finished for PR 23205 at commit e5f5774.

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

@srowen
Copy link
Member

srowen commented Dec 3, 2018

Merging as a follow up to #21688

@asfgit asfgit closed this in eebb940 Dec 3, 2018
jackylee-ch pushed a commit to jackylee-ch/spark that referenced this pull request Feb 18, 2019
…mpty table when no data is present

Task Summary Metrics Table on Stage Page shows empty table when no data is present instead of showing a message.

## What changes were proposed in this pull request?

Added a custom message to show on the task summary metrics table as well as executor summary table when no data is present.

## How was this patch tested?

**Before:**

![49335550-29277d00-f615-11e8-8e62-a953e76bcebf](https://user-images.githubusercontent.com/22228190/49361520-425a2780-f702-11e8-8df4-08862ab6ceb8.png)

**After:**

<img width="1413" alt="screen shot 2018-12-03 at 1 56 09 pm" src="https://user-images.githubusercontent.com/22228190/49362019-8699f780-f703-11e8-93e1-d02df6572923.png">

Closes apache#23205 from pgandhi999/SPARK-26253.

Authored-by: pgandhi <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants