Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1596
Thanks @agoodm for the explanation. Actually this PR would call open method
in `RemoteInterpreterServer.open` rather than in constructor, 2 reasons
* In semantics perspective, it should be called in
`RemoteInterpreterServer.open` rather than in jobRun, and we don't need to
check whether interperer is open in jobRun every time, open should be triggered
by client rather than server.
* For another PR, I'd like to pass extra info through open method from
client to server.
IIRC, you just require `InterpreterGroup` been setup before `registerHook`,
right ? Then I think this PR would not affect that. After the PR, the sequences
will be
* createInterpreter (InterpreterGroup will also be created here)
* open interpreter
* interpret paragraph (jobRun)
---
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.
---
