[GitHub] incubator-zeppelin pull request: ZEPPELIN-31 fix REST API base pat...

2015-06-11 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/30#issuecomment-111382600 Tested. +1 --- 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 th

[GitHub] incubator-zeppelin pull request: ZEPPELIN-31 fix REST API base pat...

2015-06-11 Thread anthonycorbacho
Github user anthonycorbacho commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/30#issuecomment-111381779 +1 as well On Jun 11, 2015 11:18 PM, "CORNEAU Damien" wrote: > +1 > > — > Reply to this email directly or view it on GitHub

Re: [DISCUSS] Roadmap and first release

2015-06-11 Thread tog
Hi Moon Wish you all the best for this first release under Apache. By 1.4 I was meaning default being 1.4 currently it is 1.3.1 Cheers Guillaume On Friday, June 12, 2015, moon soo Lee wrote: > Hi, Guillaume > > Thanks for asking. > > While it's our trial of release and Zeppelin need go through

[GitHub] incubator-zeppelin pull request: ZEPPELIN-31 fix REST API base pat...

2015-06-11 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/30#issuecomment-111375036 +1 --- 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 featu

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-35] Icons should have t...

2015-06-11 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/34#issuecomment-111373582 +1 for Merge! --- 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

Re: [DISCUSS] Roadmap and first release

2015-06-11 Thread moon soo Lee
Hi, Guillaume Thanks for asking. While it's our trial of release and Zeppelin need go through the Apache release process, we might/might not face some unexpected issue. So i think i couldn't foreseen accurately, but i hope we can make a release in Jun. Zeppelin is supporting spark 1.4 since http

Re: [DISCUSS] Roadmap and first release

2015-06-11 Thread tog
Hi Moon, Any foreseen date for 0.5? Will it rely on spark 1.4 which is now out? Cheers Guillaume On Friday, June 12, 2015, moon soo Lee wrote: > Hi, > > Once we have release branch, 'branch-0.5' then we can working on making a > release inside of that branch. And then we'll create a tag for th

Re: [DISCUSS] Roadmap and first release

2015-06-11 Thread moon soo Lee
Hi, Once we have release branch, 'branch-0.5' then we can working on making a release inside of that branch. And then we'll create a tag for the release. Unless proposed code is for only specific branch, we can always make PR to master branch, and we can merge PR selectively into relevant branche

Re: [DISCUSS] Roadmap and first release

2015-06-11 Thread Corneau Damien
Hi, So when would the branch-0.5 ship as a release? Would we make PR in that branch before we release, or would that branch be the release? On Fri, Jun 12, 2015 at 2:13 PM, moon soo Lee wrote: > Hi, > > I'd like to open a discussion about roadmap of Zeppelin. > > I have created initial draft he

[DISCUSS] Roadmap and first release

2015-06-11 Thread moon soo Lee
Hi, I'd like to open a discussion about roadmap of Zeppelin. I have created initial draft here https://cwiki.apache.org/confluence/display/ZEPPELIN/Zeppelin+Roadmap based on requests from online/offline so far. Please feel free to give an idea to shape roadmap of Zeppelin. And i think we can sta

[GitHub] incubator-zeppelin pull request: Zeppelin-web Spring Cleaning

2015-06-11 Thread felizbear
Github user felizbear commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/56#issuecomment-111355208 It's okay. I just have an aversion to `this` and try to eliminate it from my code. Up to you. --- If your project is set up for it, you can reply to this e

[GitHub] incubator-zeppelin pull request: Zeppelin-web Spring Cleaning

2015-06-11 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/56#issuecomment-111353057 Yeah, but like your link show, services while being a singleton (just like everything in angular) are usually use as public functions only (transformation

[GitHub] incubator-zeppelin pull request: Zeppelin-web Spring Cleaning

2015-06-11 Thread felizbear
Github user felizbear commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/56#issuecomment-111350416 @corneadoug I skimmed through the last three commits. It looks like you did a nice job. Using `controller as` pattern is definitely an improvement. The othe

Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add interpreter for...

2015-06-11 Thread Andrey Gura
Moon, Can I join to you and provide patch for ignite branch? I believe it faster way to finish Zepplein-Ignite integration. Thanks, Andrey On Tue, Jun 9, 2015 at 4:52 AM, moon soo Lee wrote: > Hi, > > Thanks for the feedback. > > I was thinking it make sense to have one connection address for

[GitHub] incubator-zeppelin pull request: ZEPPELIN-98 Fix YARN artifacts ve...

2015-06-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/97 --- 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 feat

[GitHub] incubator-zeppelin pull request: ZEPPELIN-74 Change interpreter se...

2015-06-11 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/96#issuecomment-111034114 all test are passed but travis is somehow goes to after_failure lifecycle. --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-zeppelin pull request: ZEPPELIN-98 Fix YARN artifacts ve...

2015-06-11 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/97#issuecomment-111033980 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 fea

[GitHub] incubator-zeppelin pull request: Add Scala utility functions for d...

2015-06-11 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/80#issuecomment-111033924 @doanduyhai Also i'd like to know if you're planning to add more commit here for Java to Scala and some improvements. Or if you prefer to merge it firs

[GitHub] incubator-zeppelin pull request: Adding PySparkInterpreterTest for...

2015-06-11 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/98#issuecomment-111031487 Awesome. Really great work. FYI, Recently test for pyspark has been added here https://github.com/apache/incubator-zeppelin/blob/master/zeppelin-se

[jira] [Created] (ZEPPELIN-103) Text editor disappear after moving from a report style notebook

2015-06-11 Thread CORNEAU Damien (JIRA)
CORNEAU Damien created ZEPPELIN-103: --- Summary: Text editor disappear after moving from a report style notebook Key: ZEPPELIN-103 URL: https://issues.apache.org/jira/browse/ZEPPELIN-103 Project: Zepp

[jira] [Created] (ZEPPELIN-102) Websocket doesn't update the query

2015-06-11 Thread CORNEAU Damien (JIRA)
CORNEAU Damien created ZEPPELIN-102: --- Summary: Websocket doesn't update the query Key: ZEPPELIN-102 URL: https://issues.apache.org/jira/browse/ZEPPELIN-102 Project: Zeppelin Issue Type: Bug