[GitHub] zeppelin pull request #2668: ZEPPELIN-3051. Support Interpreter Process Reco...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/2668


---

[GitHub] zeppelin pull request #2668: ZEPPELIN-3051. Support Interpreter Process Reco...

[zjffdu]

Github user zjffdu closed the pull request at:

https://github.com/apache/zeppelin/pull/2668


---

[GitHub] zeppelin pull request #2668: ZEPPELIN-3051. Support Interpreter Process Reco...

[zjffdu]

GitHub user zjffdu reopened a pull request:

https://github.com/apache/zeppelin/pull/2668

ZEPPELIN-3051. Support Interpreter Process Recovery

### What is this PR for?
This PR is for the purpose of recover running interpreter process when 
zeppelin server is restarted. This would be useful when restarting zeppelin 
without interrupt current running interpreter processes, should be useful when 
admin do maintenance or upgrading. 

Interface `RecoveryStorage` is used for storing the information of running 
interpreter process.
Currently it only has one implementation `FileSystemRecoveryStorage`, other 
implementation could be done later (such as zookeeper based). 
`InterpreterLauncher` is the component where to recover the running interpreter 
process.

Test:
* RecoveryTest.java
* FileSystemRecoveryStorageTest.java

Design Doc:


https://docs.google.com/document/d/1Plm3Hd40aGdNaXmjdsoY4ek3f-gTijTMGMkNjAZN39Y/edit?usp=sharing

### What type of PR is it?
[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3051

### How should this be tested?
Unit test & Integration Test is added. Also manually verified. 

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-3051

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2668.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2668


commit c0b7dd987c2ad64d8ccb8e2ae2434d1dc0bd39bf
Author: Jeff Zhang 
Date:   2017-11-14T13:48:06Z

ZEPPELIN-3051. Support Interpreter Process Recovery

commit de69c678cdd96ac348a44bc5d6f0348246b078e8
Author: Jeff Zhang 
Date:   2017-11-16T00:26:11Z

address comments




---

[GitHub] zeppelin pull request #2668: ZEPPELIN-3051. Support Interpreter Process Reco...

[zjffdu]

GitHub user zjffdu opened a pull request:

https://github.com/apache/zeppelin/pull/2668

ZEPPELIN-3051. Support Interpreter Process Recovery

### What is this PR for?
This PR is for the purpose of recover running interpreter process when 
zeppelin server is restarted. This would be useful when restarting zeppelin 
without interrupt current running interpreter processes, should be useful when 
admin do maintenance or upgrading. 

Interface `RecoveryStorage` is used for storing the information of running 
interpreter process.
Currently it only has one implementation `FileSystemRecoveryStorage`, other 
implementation could be done later (such as zookeeper based). 
`InterpreterLauncher` is the component where to recover the running interpreter 
process.

### What type of PR is it?
[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3051

### How should this be tested?
Unit test & Integration Test is added. Also manually verified. 

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-3051

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2668.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2668


commit c0b7dd987c2ad64d8ccb8e2ae2434d1dc0bd39bf
Author: Jeff Zhang 
Date:   2017-11-14T13:48:06Z

ZEPPELIN-3051. Support Interpreter Process Recovery




---