Skip to content

[SPARK-32288][UI] Add exception summary for failed tasks in stage page #31870

Closed
warrenzhu25 wants to merge 2 commits intoapache:masterfrom
warrenzhu25:exception
Closed

[SPARK-32288][UI] Add exception summary for failed tasks in stage page #31870
warrenzhu25 wants to merge 2 commits intoapache:masterfrom
warrenzhu25:exception

Conversation

@warrenzhu25
Copy link
Contributor

What changes were proposed in this pull request?

Add exception summary table for failed tasks in stage page.
Before:
image

After:
image

Why are the changes needed?

When there're many task failure during one stage, it's hard to find failure pattern such as aggregation task failure by exception type and message. If we have such information, we can easily know which type of exception of failure is the root cause of stage failure.

Does this PR introduce any user-facing change?

Yes

How was this patch tested?

Added UT in UISeleniumSuite

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

Copy link
Member

@Ngone51 Ngone51 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks useful. cc @gengliangwang @sarutak

@github-actions
Copy link

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!

@github-actions github-actions bot added the Stale label Jun 29, 2021
@github-actions github-actions bot closed this Jun 30, 2021
@warrenzhu25
Copy link
Contributor Author

@gengliangwang @sarutak @tgravescs Could you help take a look and remove stale label?

@warrenzhu25 warrenzhu25 deleted the exception branch June 11, 2022 17:25
@warrenzhu25 warrenzhu25 restored the exception branch September 20, 2022 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants