Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/6672#issuecomment-109474613
Awesome! Screenshot looking pretty good. Here are some nit picky comments.
- RDD, Receiver Blocks, these titles are same size as "Storage" at the top.
Should be smaller.
- Address / Exec ID format is little confusing. Might be a good idea to
ignore the exec ID. If some one needs to relate it to the executor, they can
use the hostname to find it themselves on the executors page.
- How will it look if one of the 2 copies of the blocks fall off from the
executor?
Related
- We should not drop blocks from the UI, because the Storage page is meant
to be a true representation of the what is going on in the cluster. Instead we
could make multiple collapsible sections, one for each input stream
- Let's not put the broadcast blocks in the page yet, because we are not
even sure whether its worth putting those blocks, whether people even need to
debug that. And whether showing the individual pieces is worth it or not. Lets
keep those questions outside the scope of this PR, and address them in a
separate PR.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
