Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/228#issuecomment-133769850
@akhiluanandh Thanks for the contribution.
Can SparkInterpreter handle restart SparkContext locally without adding a
new method in Interpreter class? ie. restart SparkContext only, not whole
interpreter process.
Reasons i think restart only SparkContext is is better,
* We can keep Interpreter class simple. One less thing to understand to
implement new interpreter for Zeppelin.
* When dependency loader loads some libraries, restarting SparkContext can
reuse loaded libraries, but restarting interpreter process can not use these
libraries, in current architecture. So user may have different libraries loaded
in this environment after restart and it can be very confusing, especially when
restarting is done silently.
What do you think @akhiluanandh ?
---
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.
---
