[GitHub] incubator-zeppelin pull request: Auto-restart interpreters on cron...

2015-12-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/557 --- 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 fea

[GitHub] incubator-zeppelin pull request: Auto-restart interpreters on cron...

2015-12-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-167616036 Tested and LGTM. Merge if there're no more discussions --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-zeppelin pull request: Auto-restart interpreters on cron...

2015-12-27 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-167461320 @Leemoonsoo Thank you for making clear. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] incubator-zeppelin pull request: Auto-restart interpreters on cron...

2015-12-25 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-167280810 @Leemoonsoo I took a look at previous logs, and its behavior was somewhat strange. I'll file an issue regarding strange behavior with logs. (Woul

[GitHub] incubator-zeppelin pull request: Auto-restart interpreters on cron...

2015-12-25 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-167275152 @astroshim right, I think separated interpreter process for cron job or similar would be a good approach for this issues. I know that there are places wi

[GitHub] incubator-zeppelin pull request: Auto-restart interpreters on cron...

2015-12-25 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-167243619 Hello. @felixcheung You're right.. The interpreter process is shared now but I think interpreter process will be separated as user option. what d

[GitHub] incubator-zeppelin pull request: Auto-restart interpreters on cron...

2015-12-24 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-167172536 @Leemoonsoo It occurred while I was experimenting several things together so I'm not 100% sure that we can't handle interpreter via UI. I'll try

[GitHub] incubator-zeppelin pull request: Auto-restart interpreters on cron...

2015-12-24 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-167169459 I think interpreter restart should abort pending / running jobs. If it raise `Scheduler already terminated` need to be fixed. @HeartSaVioR do you mind

[GitHub] incubator-zeppelin pull request: Auto-restart interpreters on cron...

2015-12-24 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-167163100 Actually I was trying to restart the interpreter first for each job run via REST API, but gave up because it throws ```Scheduler already terminated``` whe

[GitHub] incubator-zeppelin pull request: Auto-restart interpreters on cron...

2015-12-24 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-167152568 Would it be possible the interpreter restart happens in the middle of another running notebook if more one are scheduled to around the same time? Inte