[GitHub] [zeppelin] liuxunorg opened a new pull request #3389: [ZEPPELIN-4209] Fixed the ClusterInterpreterLauncher code bug

[GitBox <gi...@apache.org>]

liuxunorg opened a new pull request #3389: [ZEPPELIN-4209] Fixed the ClusterInterpreterLauncher code bug
URL: https://github.com/apache/zeppelin/pull/3389
 
 
   ### What is this PR for?
   1. **ClusterInterpreterLauncher can not listen cluster events**
   In Cluster mode, The `ClusterInterpreterLauncher` needs to listen for cluster events, Accept requests from other zeppelin servers to create interpreter processes.
   Since the `ClusterInterpreterLauncher` is lazy, dynamically generated, So in cluster mode, when the zeppelin service starts, Create a `ClusterInterpreterLauncher` object, This allows the `ClusterInterpreterLauncher` to listen for cluster events.
   
   2. **Replace `RemoteInterpreterManagedProcess` with `ClusterInterpreterProcess` for local interpreter process creation**
   Because in cluster mode, `RemoteInterpreterServer` performs service discovery by registering thrift's `ip` and `port` into the cluster metadata, not through the thrift interface `callback`.
   
   
   ### What type of PR is it?
   [Bug Fix]
   
   
   ### What is the Jira issue?
   * https://issues.apache.org/jira/browse/ZEPPELIN-4209
   
   ### How should this be tested?
   * [CI Pass](https://travis-ci.org/liuxunorg/zeppelin/builds/552320897)
   
   ### Screenshots (if appropriate)
   
   ### Questions:
   * Does the licenses files need update?
   * Is there breaking changes for older versions?
   * Does this needs documentation?
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services