Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19981
> I also not sure why we need a plugin interface for live UI.
So your solution to that is to have completely separate code for both
cases? I really prefer to have a single place to go to to understand how the
listeners and the UI are initialized, even if the SQL plugin implementation is
sub-optimal.
Again, you're stating a matter of preference, but you haven't explained
what problem this is causing. Not using the interface doesn't mean you get rid
of the conditionals, it just means the conditionals are implied by having the
code that installs the listener and UI for live applications live in a
different place.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
