Github user tgravescs commented on the issue:

    https://github.com/apache/spark/pull/14617
  
    while I kind of like the hover because it doesn't clutter the page, it does 
bring up a couple concerns:
    - user can't sort by them
    - user might not know to hover (none of the other pages that I know of show 
useful values like this)
    
    The latter one is perhaps the one that is more concerning.  Unless we can 
highlight it or somehow draw the users attention to it they will not know its 
there.   But the first one is also pretty valid. If I have thousands of 
executors and looking for one that is out of off heap memory, its pretty 
difficult to hover over all of them.
    
    I don't see any changes to the Storage UI page, do we want to add something 
there 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

Reply via email to