[GitHub] incubator-zeppelin pull request: ZEPPELIN-385 Read-only mode for z...

2015-11-16 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/389#issuecomment-157302094 I'm a bit late for the party, However I would prefer the ``isReadOnly()`` part in zeppelin-web to be handled as a DataFactory instead of storing it in

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-421] Remove deprecated ...

2015-11-16 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/435#issuecomment-157299624 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 feat

[GitHub] incubator-zeppelin pull request: ZEPPELIN-385 Read-only mode for z...

2015-11-16 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/389#issuecomment-157299400 @Leemoonsoo I think we should take a care of a style of `import *`. What do you think of it? This looks minor for now but it needs to keep it. --- If your

[GitHub] incubator-zeppelin pull request: ZEPPELIN-412 Documentation based ...

2015-11-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/430#issuecomment-157279509 Merging #430 and #431 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 GitH

[GitHub] incubator-zeppelin pull request: ZEPPELIN-427: Upgrade to Apache T...

2015-11-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/439#issuecomment-157278977 I think the last CI fail is irrelevant. Looks good to me. --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] incubator-zeppelin pull request: ZEPPELIN-293 notebook execution r...

2015-11-16 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/incubator-zeppelin/pull/441 ZEPPELIN-293 notebook execution results leaking to dashboard page Original PR https://github.com/apache/incubator-zeppelin/pull/293 When we run a paragraph, and without it

[GitHub] incubator-zeppelin pull request: ZEPPELIN-293 notebook execution r...

2015-11-16 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/293 --- 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] incubator-zeppelin pull request: ZEPPELIN-293 notebook execution r...

2015-11-16 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/293#issuecomment-157274482 Doing a new PR now, and closing this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] incubator-zeppelin pull request: ZEPPELIN-426 - Allowing new parag...

2015-11-16 Thread corneadoug
GitHub user corneadoug opened a pull request: https://github.com/apache/incubator-zeppelin/pull/440 ZEPPELIN-426 - Allowing new paragraph under the last one Adding a paragraph under the last one of the notebook was not allowed. I tested a lot of different cases: * While runni

[GitHub] incubator-zeppelin pull request: ZEPPELIN-426 - Allowing new parag...

2015-11-16 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/440#issuecomment-157262507 Ready for tests/reviews --- 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 do

[GitHub] incubator-zeppelin pull request: ZEPPELIN-414 - Contain result ins...

2015-11-16 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/436#issuecomment-157248651 Merging if there is no more discussions --- 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: ZEPPELIN-293 notebook execution r...

2015-11-16 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/293#issuecomment-157248559 @Leemoonsoo if you still have that branch locally, could you do a new PR? --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-zeppelin pull request: ZEPPELIN-431 - Show Karma test re...

2015-11-16 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/390#issuecomment-157247769 Just changed the scope of this PR, it is now ready to be merged. Code coverage is still there, although the report path changed from inside ``zeppelin-

[GitHub] incubator-zeppelin pull request: ZEPPELIN-427: Upgrade to Apache T...

2015-11-16 Thread blrunner
Github user blrunner commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/439#issuecomment-157246146 I made two CI builds and one of them succeeded as following: https://travis-ci.org/apache/incubator-zeppelin/builds/91511236 --- If your project is set

[jira] [Created] (ZEPPELIN-431) Karma test result not showing

2015-11-16 Thread CORNEAU Damien (JIRA)
CORNEAU Damien created ZEPPELIN-431: --- Summary: Karma test result not showing Key: ZEPPELIN-431 URL: https://issues.apache.org/jira/browse/ZEPPELIN-431 Project: Zeppelin Issue Type: Bug

[GitHub] incubator-zeppelin pull request: ZEPPELIN-430 - Remove deprecated ...

2015-11-16 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/438#issuecomment-157235467 Will merge if there is no more comments --- 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: ZEPPELIN-430 - Remove deprecated ...

2015-11-16 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/438#issuecomment-157235437 @Leemoonsoo issue created here: https://issues.apache.org/jira/browse/ZEPPELIN-430 --- If your project is set up for it, you can reply to this email a

[jira] [Created] (ZEPPELIN-430) Replace grunt-autoprefixer for frunt-postcss

2015-11-16 Thread CORNEAU Damien (JIRA)
CORNEAU Damien created ZEPPELIN-430: --- Summary: Replace grunt-autoprefixer for frunt-postcss Key: ZEPPELIN-430 URL: https://issues.apache.org/jira/browse/ZEPPELIN-430 Project: Zeppelin Issue

[GitHub] incubator-zeppelin pull request: ZEPPELIN-427: Upgrade to Apache T...

2015-11-16 Thread blrunner
Github user blrunner commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/439#issuecomment-15726 Hi @Leemoonsoo Thank you for your quick feedback. I've just added some unit test cases. The CI build will run soon. :) --- If your project is set

[GitHub] incubator-zeppelin pull request: R Interpreter for Zeppelin

2015-11-16 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-157203411 The current push should resolve some issues with changes in the Spark-Zeppelin interface that had created issues for users, as well as support for 1.5.1.

Re: why zeppelin SparkInterpreter use FIFOScheduler

2015-11-16 Thread Rohit Agarwal
Hey Pranav, Did you make any progress on this? -- Rohit On Sunday, August 16, 2015, moon soo Lee wrote: > Pranav, proposal looks awesome! > > I have a question and feedback, > > You said you tested 1,2 and 3. To create SparkIMain per notebook, you need > information of notebook id. Did you get

Re: [RESULT] [VOTE] Release Apache Zeppelin (incubating) 0.5.5-incubating (RC3)

2015-11-16 Thread Jonathan Kelly
Ah, I see. I hadn't noticed that your other voting email was for a different list, and I didn't realize that there was a requirement for a second level of voting for incubating projects. Thanks for the clarification. ~ Jonathan On Mon, Nov 16, 2015 at 12:19 PM, tog wrote: > OK Moon Thanks for t

[GitHub] incubator-zeppelin pull request: ZEPPELIN-427: Upgrade to Apache T...

2015-11-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/439#issuecomment-157184867 Thanks for nice upgrade. CI looks like failing with the other reason, but could you try trigger CI? close / reopen this PR will trigger the CI.

[GitHub] incubator-zeppelin pull request: R Interpreter for Zeppelin

2015-11-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-157180281 @elbamos Please do not blame people in your PR. That's out of your PR's scope. And I thought you and Felix both agreed to move forward, If you think t

Re: [GitHub] incubator-zeppelin pull request: R Interpreter for Zeppelin

2015-11-16 Thread Eran Witkon
Are you all sure this is how we want to run an open source project? I can say I wrote this code but it doesn't make it right. When we contribute code to the open source community we prefer the better good then our own private ego. I have been working with the zeppelin project since its beginning a

[GitHub] incubator-zeppelin pull request: R Interpreter for Zeppelin

2015-11-16 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-157158373 To anyone who may be coming here from today's blog post and confused by these comments about the current status: The RInterpreter for zeppelin works

Re: [RESULT] [VOTE] Release Apache Zeppelin (incubating) 0.5.5-incubating (RC3)

2015-11-16 Thread tog
OK Moon Thanks for the information, I was not aware of that step. Cheers Guillaume On Nov 16, 2015 8:10 PM, "moon soo Lee" wrote: > Hi, > > I think it's my bad that not explaining well enough about the next step. > > While Zeppelin is in incubation, after vote passed in dev@zeppelin, > another >

Re: [RESULT] [VOTE] Release Apache Zeppelin (incubating) 0.5.5-incubating (RC3)

2015-11-16 Thread moon soo Lee
Hi, I think it's my bad that not explaining well enough about the next step. While Zeppelin is in incubation, after vote passed in dev@zeppelin, another vote is required in general@incubator with IPMC. Here's related information about release process in incubation. http://incubator.apache.org/in

Re: [RESULT] [VOTE] Release Apache Zeppelin (incubating) 0.5.5-incubating (RC3)

2015-11-16 Thread Jonathan Kelly
Moon, I think I may have the same confusion as Guillaume, as you posted a message on this same thread a couple of days ago saying that the vote for RC3 has passed already, and the email contained a link to http://mail-archives.apache.org/mod_mbox/incubator-zeppelin-dev/201511.mbox/%3CCALf24sboheQd

Re: Help regarding finding a functionality in the source code

2015-11-16 Thread moon soo Lee
Hi Kunal, Thanks for using Zeppelin. Currently, there're PR https://github.com/apache/incubator-zeppelin/pull/437 that adds dynamic form feature into pyspark. When Implementing an Interpreter, getFormType() abstract method need to be implemented. https://github.com/apache/incubator-zeppelin/blob

Re: [RESULT] [VOTE] Release Apache Zeppelin (incubating) 0.5.5-incubating (RC3)

2015-11-16 Thread moon soo Lee
Hi Guillaume, Thanks for asking. 0.5.5-incubating is in vote at general mailing list http://mail-archives.apache.org/mod_mbox/incubator-general/201511.mbox/%3CCALf24sZin774uuZ%2BftOLV4pBf8egWyJRAMK3fscyh_kcQXZaig%40mail.gmail.com%3E Once vote is passed at general@incubator and packages are sync

Re: [RESULT] [VOTE] Release Apache Zeppelin (incubating) 0.5.5-incubating (RC3)

2015-11-16 Thread tog
Hi Moon Do we have an official annonce now on user mailing list following the vote? Cheers Guillaume On Nov 14, 2015 12:36 PM, "moon soo Lee" wrote: > The vote passes with 7 binding +1 votes, 9 non-binding +1 votes, and no +0 > or -1 votes. Thanks for everyone who verified rc and voted. > > > +

[GitHub] incubator-zeppelin pull request: Zeppelin Flink Spark tutorial

2015-11-16 Thread epugh
Github user epugh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/418#issuecomment-157136166 @rawkintrevo I found it very valuable... I'd never set up Spark or Flink, and your tutorial assumed a much lower degree of knowledge. --- If your project

[GitHub] incubator-zeppelin pull request: ZEPPELIN-427: Upgrade to Apache T...

2015-11-16 Thread blrunner
GitHub user blrunner opened a pull request: https://github.com/apache/incubator-zeppelin/pull/439 ZEPPELIN-427: Upgrade to Apache TAJO 0.11.0 I bumped up Tajo to 0.11.0. And I enabled ```cancel()``` because Tajo supports it since 0.11.0 release. For the reference, this patch ran su

[GitHub] incubator-zeppelin pull request: R Interpreter for Zeppelin

2015-11-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-157040049 And if you are not comfortable working with Zeppelin community, not because of you're wrong or Zeppelin community is wrong, but maybe just because of the

Help regarding finding a functionality in the source code

2015-11-16 Thread kunal chelani
Hi Guys, I am Kunal. I have been using zeppelin for some time now. I have made a few changes in the code as well for my personal modifications and I am stuck at one particular requirement. Here goes my doubt : If we use the %sql of %hive interpreter and write something with the pattern '${}' , a

[GitHub] incubator-zeppelin pull request: R Interpreter for Zeppelin

2015-11-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-157017167 Sometimes people can argue then I see the passion. But when people don't respect each other, it easily become blaming and fighting. Let's try to n

[GitHub] incubator-zeppelin pull request: ZEPPELIN-422 Add missing Apache H...

2015-11-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/433 --- 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: Remove deprecated grunt-autoprefi...

2015-11-16 Thread rconline
Github user rconline commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/438#issuecomment-157002527 Looks 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. If your project does not

[GitHub] incubator-zeppelin pull request: ZEPPELIN-385 Read-only mode for z...

2015-11-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/389#issuecomment-156999761 Merging 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 as well. I

[GitHub] incubator-zeppelin pull request: Remove deprecated grunt-autoprefi...

2015-11-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/438#issuecomment-156998827 Looks good to me. Do you mind creating a corresponding issue? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-421] Remove deprecated ...

2015-11-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/435#issuecomment-156998170 Looks 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. If your project does not

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-428] Support Python pro...

2015-11-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/437#issuecomment-156997464 Working really well. Looks 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-414 - Contain result ins...

2015-11-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/436#issuecomment-156995056 I have a notebook that have result larger than paragraph width. Tested with this PR and i can see it is fixed. Looks good to me. --- If your pro

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-420] Improve notebook c...

2015-11-16 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/432#issuecomment-156979502 Nice improvement! 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

[jira] [Created] (ZEPPELIN-429) Replace Explicit type with diamond type

2015-11-16 Thread Dongkyu Hwangbo (JIRA)
Dongkyu Hwangbo created ZEPPELIN-429: Summary: Replace Explicit type with diamond type Key: ZEPPELIN-429 URL: https://issues.apache.org/jira/browse/ZEPPELIN-429 Project: Zeppelin Issue Ty

[GitHub] incubator-zeppelin pull request: Remove deprecated grunt-autoprefi...

2015-11-16 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/438#issuecomment-156950726 Ready For Reviews, Merge and more :D --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] incubator-zeppelin pull request: Remove deprecated grunt-autoprefi...

2015-11-16 Thread corneadoug
GitHub user corneadoug opened a pull request: https://github.com/apache/incubator-zeppelin/pull/438 Remove deprecated grunt-autoprefixer for grunt-postcss Making a switch off grunt plugin since grunt-autoprefixer is deprecated, and its version of autoprefixer is old. You can merge