Github user vanzin commented on the issue:

    https://github.com/apache/spark/pull/14204
  
    So, if I understand this correctly, in non-standalone mode you'll have a 
completely empty column in the UI taking up space?
    
    Why is it important to collect the worker URL (not the UI address)? There's 
nothing users can do with that.
    
    I'm also not a fan of exposing things that are particular to one specific 
cluster manager in the REST API.
    
    I'd be a little more ok with this patch if it hid that information from the 
UI when it's not available. 
    
    But in fact I don't really see much value in exposing that link. To me, it 
just clutters up the UI, so you might as well just have a link to the Master UI 
somewhere and tell users to find the worker there if they need it.


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