[GitHub] zeppelin issue #1476: [WIP] ZEPPELIN-335. Pig Interpreter

2016-10-08 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1476 @felixcheung please check `pig.md` --- 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 #1487: [ZEPPELIN-1520] Change test environment to use the loc...

2016-10-08 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1487 Should we provide the option to allow user to use other modes ? Otherwise it seems the scripts under folder `test` can be removed as well. --- If your project is set up for it, you can reply to t

[GitHub] zeppelin issue #1487: [ZEPPELIN-1520] Change test environment to use the loc...

2016-10-08 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1487 And think again, how about making the local mode as the default mode for local dev (non-travis enviroment), and use standalone mode for travis ? Because local mode success doesn't mean other modes

[GitHub] zeppelin pull request #1496: [Zeppelin-907] Apply new mechanism to AngularIn...

2016-10-08 Thread vectorijk
GitHub user vectorijk opened a pull request: https://github.com/apache/zeppelin/pull/1496 [Zeppelin-907] Apply new mechanism to AngularInterpreter ### What is this PR for? This PR applies the new interpreter registration mechanism to Angular interpreter. ### What type o

Re: Refactor time !

2016-10-08 Thread Eric Charles
On 04/10/16 12:54, Anthony Corbacho wrote: You made my day, this is the kind of email i really like !! I think its a great idea and i am willing to spend sometime on it. I also want to move to a DI (guice) architecture , let me know what you think about it. A PR is opened for Guice DI. If som

[GitHub] zeppelin issue #1496: [Zeppelin-907] Apply new mechanism to AngularInterpret...

2016-10-08 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue: https://github.com/apache/zeppelin/pull/1496 @vectorijk thank you for the contribution the change look good to me --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] zeppelin pull request #1497: [ZEPPELIN-1233] Support Redis interpreter

2016-10-08 Thread darionyaphet
GitHub user darionyaphet opened a pull request: https://github.com/apache/zeppelin/pull/1497 [ZEPPELIN-1233] Support Redis interpreter ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out the contributing

Re: How Apache Zeppelin runs a paragraph

2016-10-08 Thread Felix Cheung
Great post! On Tue, Oct 4, 2016 at 8:56 PM -0700, "Jongyoul Lee" mailto:jongy...@gmail.com>> wrote: Hello DuyHai, Thanks for the fixing the typo. I've fixed it. Concerning the debugging, I think writing posts or updating wiki would be better. I'm willing to write a simple post for that re

Re: [DISCUSS] Zeppelin 0.6.2 release

2016-10-08 Thread Felix Cheung
Seems like we have a couple of good fixes. +1 for another release On Fri, Oct 7, 2016 at 11:53 PM -0700, "Ahyoung Ryu" mailto:ahyoung...@apache.org>> wrote: +1 On Sat, Oct 8, 2016 at 1:45 PM, Prabhjyot Singh wrote: > +1 > > On 8 Oct 2016 5:42 a.m., "Hyung Sung Shim" wrote: > > > +1 > >

[GitHub] zeppelin issue #1496: [Zeppelin-907] Apply new mechanism to AngularInterpret...

2016-10-08 Thread vectorijk
Github user vectorijk commented on the issue: https://github.com/apache/zeppelin/pull/1496 @anthonycorbacho Thanks for your quick response. --- 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 thi

Re: How Apache Zeppelin runs a paragraph

2016-10-08 Thread Jeff Zhang
Thanks Jougyoul for the effort. This is really helpful for new contributors. Several comments. 1. Seems we didn't use noteId/notebookId correctly in code, do we have ticket to track that ? Because the sample code is conflicted with the definition of note/notebook in the post, it might confuse use

[GitHub] zeppelin pull request #1498: [DOCS] Fix dead link in docs/datasource_authori...

2016-10-08 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/zeppelin/pull/1498 [DOCS] Fix dead link in docs/datasource_authorization.md ### What is this PR for? A link for `the community interpreters` in [here](http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/security/

[GitHub] zeppelin issue #1498: [DOCS] Fix dead link in docs/datasource_authorization....

2016-10-08 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue: https://github.com/apache/zeppelin/pull/1498 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 and wishes so, or if

[GitHub] zeppelin issue #1498: [DOCS] Fix dead link in docs/datasource_authorization....

2016-10-08 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1498 @anthonycorbacho @minahlee Thank you for the prompt response! @minahlee Yeah I didn't thought about that. Reverted it :) --- If your project is set up for it, you can reply to this email and

[GitHub] zeppelin issue #1498: [DOCS] Fix dead link in docs/datasource_authorization....

2016-10-08 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1498 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 and wishes so, or if the fe

[GitHub] zeppelin issue #1498: [DOCS] Fix dead link in docs/datasource_authorization....

2016-10-08 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1498 CI is failed at `SCALA_VER="2.10" SPARK_VER="1.4.1" HADOOP_VER="2.3"` with ``` No output has been received in the last 10 minutes, this potentially indicates a stalled build or somethin

[GitHub] zeppelin issue #1480: [ZEPPELIN-1489] Remove jdbc-like interpreter example p...

2016-10-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1480 @AhyoungRyu Great work! It definitely helps using JDBC datasource in Zeppelin. CI failure is not related. LGTM and merge if there're no further comments. --- If your project is se

[GitHub] zeppelin issue #133: [ZEPPELIN-127] Fix issue raised by ZEPPELIN-127 related...

2016-10-08 Thread doanduyhai
Github user doanduyhai commented on the issue: https://github.com/apache/zeppelin/pull/133 @Leemoonsoo The fix is using pretty standard Scala (`object xxx extends Serializable`) and not impacted by Scala 2.11 breaking changes --- If your project is set up for it, you can re