Github user kayousterhout commented on the pull request:

    https://github.com/apache/spark/pull/9487#issuecomment-155670350
  
    I'd prefer your original approach to this, because in either case, it takes 
up one line of text in the default view, but with this, it takes up one line of 
text *and* you have to click on that line to see the metrics.
    
    @andrewor14 @srowen what about putting the summary ("Locality summary: 
process local: 3 tasks; dode local: 6 tasks; ..." under the "Summary metrics" 
heading and above the table (and always displayed)?
    
    @jbonofre might be a good idea to hold off on changing the code until 
there's consensus on the right approach


---
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]

Reply via email to