Github user cloud-fan commented on the issue:

    https://github.com/apache/spark/pull/19981
  
    I don't think `AppStatusPlugin` is well designed to support both live and 
history server. The fact that `SQLAppStatusPlugin` needs flags to decide to 
register the listener in `setupListeners` or `setupUI` looks pretty hacky.
    
    I also not sure why we need a plugin interface for live UI. The SQL and 
Streaming UI are implemented pretty well and clean. I do agree that an 
interface for history server is very useful, and that's why we have 
`SparkHistoryListenerFactory` before.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to