You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@zeppelin.apache.org by Leemoonsoo <gi...@git.apache.org> on 2017/11/20 18:47:51 UTC

[GitHub] zeppelin issue #2681: [ZEPPELIN-2995] "auto-restart interpreter on cron exec...

Github user Leemoonsoo commented on the issue:

    https://github.com/apache/zeppelin/pull/2681
  
    Thanks @kjmrknsn for the fix. It looks great to me!
    
    If we can add a automated test, like [testAutoRestartInterpreterAfterSchedule ](https://github.com/apache/zeppelin/blob/master/zeppelin-zengine/src/test/java/org/apache/zeppelin/notebook/NotebookTest.java#L405), it'll help make sure any future change doesn't break this behavior.
    
    Do you think you can add a test? Let me know if you need help on adding test!


---