[GitHub] incubator-zeppelin pull request: ZEPPELIN-528 REST API: add "progr...

2015-12-24 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/563#issuecomment-167065178 Great. Looks good to me. 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 a

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-391] Keyboard shortcut

2015-12-24 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/569#issuecomment-167130741 Please review --- 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 hav

[GitHub] incubator-zeppelin pull request: ZEPPELIN-528 REST API: add "progr...

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

[GitHub] incubator-zeppelin pull request: Fix z-index of control button too...

2015-12-24 Thread minahlee
GitHub user minahlee opened a pull request: https://github.com/apache/incubator-zeppelin/pull/570 Fix z-index of control button tooltip ### What is this PR for? Fix z-index for control button tooltip ### What type of PR is it? Bug Fix ### Is there a relevant

[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: ZEPPELIN-511 REST API: Insert / R...

2015-12-24 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/550#issuecomment-167165446 Done with another upmerge. --- 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 projec

[GitHub] incubator-zeppelin pull request: Remove author tag

2015-12-24 Thread minahlee
GitHub user minahlee opened a pull request: https://github.com/apache/incubator-zeppelin/pull/571 Remove author tag ### What is this PR for? Remove author tag from java doc since Zeppelin doesn't encourage to use it ### What type of PR is it? Documentation #

[GitHub] incubator-zeppelin pull request: Remove author tag

2015-12-24 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/571#issuecomment-167165553 +1 (non-binding) --- 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

[GitHub] incubator-zeppelin pull request: Remove author tag

2015-12-24 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/571#issuecomment-167165900 LGTM --- 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 fe

[jira] [Created] (ZEPPELIN-531) Job management

2015-12-24 Thread Chae-Sung Lim (JIRA)
Chae-Sung Lim created ZEPPELIN-531: -- Summary: Job management Key: ZEPPELIN-531 URL: https://issues.apache.org/jira/browse/ZEPPELIN-531 Project: Zeppelin Issue Type: Improvement R

add Job management

2015-12-24 Thread Estail7s
Zeppelin job management. Zeppelin is not easy to have a current status and management of the entire node. In particular, in many cases Notebook is made. For this, I am ready to function with a separate page named 'JOB'. I confirm that this feature is part of the Road-map of the Zeppelin. Consider b

[GitHub] incubator-zeppelin pull request: Zeppelin-523 ] Table Header fixed...

2015-12-24 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/565#issuecomment-167169284 Looks good to me. 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

[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-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: Fix z-index of control button too...

2015-12-24 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/570#issuecomment-167178119 LGTM --- 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 fe

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-391] Keyboard shortcut

2015-12-24 Thread minahlee
Github user minahlee commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/569#issuecomment-167180651 Ctrl + c is commonly used to copy highlighted text to the clipboard so better to change. --- If your project is set up for it, you can reply to this email

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-391] Keyboard shortcut

2015-12-24 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/569#issuecomment-167180845 @minahlee You're right. Changed to Ctrl+Alt+c --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-391] Keyboard shortcut

2015-12-24 Thread minahlee
Github user minahlee commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/569#issuecomment-167180989 Ctrl + Alt + k and Ctrl + Alt + E does work on chrome, firefox but not on IE and Ctrl + Alt + 1~0,-,+ doesn't work on all three browsers --- If your pro

Re: zeppelin build - downloading spark distribution

2015-12-24 Thread Jeff Steinmetz
Ah yes - Now I recall the discussion related to ensuring -Ppyspark will work. Thank you for the reminder. J On 12/23/15, 5:34 PM, "moon soo Lee" wrote: >Hi Jeff Steinmetz, > >-Ppyspark flag downloads spark distribution. See >https://github.com/apache/incubator-zeppelin/blob/master/spark-depe

[GitHub] incubator-zeppelin pull request: Add a (local mode) Scalding Inter...

2015-12-24 Thread sriramkrishnan
Github user sriramkrishnan commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/561#issuecomment-167203417 Addressed all PR comments, except for docs, license as @bzz has suggested, which I will do next. Build is also green now. ps: I do need the ha