Github user squito commented on the pull request:
https://github.com/apache/spark/pull/6644#issuecomment-127734024
Hi @suyanNone any updates here? This is a really nice addition to help
users understand when things go wrong, but needs a little followup. As Andrew
requested, we'd like to see a screenshot of the modified UI. Also we should
limit how many removed executors we store for really long apps -- I think you
could add that in `StorageStatusListener`. Finally we need some minimal test
of the behavior, but I think that will be easy to add to `HistoryServerSuite`
and / or `UISeleniumSuite`. I can help out with parts of this as well.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
