[GitHub] zeppelin issue #1595: [ZEPPELIN-1615] - Zeppelin should be able to run witho...

2016-11-04 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue: https://github.com/apache/zeppelin/pull/1595 @Leemoonsoo thank you for the feedback, please fix the fix under PR #1604 --- 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] zeppelin issue #1595: [ZEPPELIN-1615] - Zeppelin should be able to run witho...

2016-11-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1595 Looks like zeppelin-web dev mode (`grunt serve`) has some problem after the patch. @anthonycorbacho Could you take a look? --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #1595: [ZEPPELIN-1615] - Zeppelin should be able to run witho...

2016-11-04 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue: https://github.com/apache/zeppelin/pull/1595 Hell yeah!!! I allow you to merge it :) --- 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 ha

[GitHub] zeppelin issue #1595: [ZEPPELIN-1615] - Zeppelin should be able to run witho...

2016-11-04 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1595 Finally CI is green!!! I am merging it --- 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 e

[GitHub] zeppelin issue #1595: [ZEPPELIN-1615] - Zeppelin should be able to run witho...

2016-11-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1595 Tested and it works well. 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 feature enabled

[GitHub] zeppelin issue #1595: [ZEPPELIN-1615] - Zeppelin should be able to run witho...

2016-11-04 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1595 Test fails, can you fix it please? --- 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 enabl

[GitHub] zeppelin issue #1595: [ZEPPELIN-1615] - Zeppelin should be able to run witho...

2016-11-04 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1595 tested and changes 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 feature enabled an