[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Add support for us...

2016-02-08 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/697#issuecomment-181462386 Please see these latest commits. First, is to make changes as previously discussed for documenting optional "user" parameter as well as properly handling

[GitHub] incubator-zeppelin pull request: Add new selenium test case for cr...

2016-02-08 Thread ravicodder
Github user ravicodder commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/688#issuecomment-181249777 @Leemoonsoo Modified test case to cover new paragraph by clicking + buttons between paragraph. --- If your project is set up for it, you can reply to

[GitHub] incubator-zeppelin pull request: Added Kerberos documentation

2016-02-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/640#issuecomment-181250941 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

[GitHub] incubator-zeppelin pull request: add new selenium test case for mo...

2016-02-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/687 --- 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

[GitHub] incubator-zeppelin pull request: Add new selenium test case for cr...

2016-02-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/688#issuecomment-181252300 @ravicodder Thanks for taking care. LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-zeppelin pull request: Update NVD3 library for supportin...

2016-02-08 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/704#issuecomment-181442468 Hi @camilosampedro Thank you for the contribution. We updated to 1.7.1 some time ago: #597 And we didn't use 1.8.1 on purpose since there was

[GitHub] incubator-zeppelin pull request: R and SparkR Support [WIP]

2016-02-08 Thread justsomeone1001
Github user justsomeone1001 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-181463479 @echarles It doesn't support base plotting. Do you have an example of code to show that it supports *interactive* visualizations? I believer it

Re: Zeppelin inside an OSGI container

2016-02-08 Thread Tom Barber
Thanks Alex! It was great meeting you too. Okay that makes sense, looks feasible. I'm stuck in Paris this week so if I get a spare evening I'll start investigating the server components. Thanks Tom -- Director Meteorite.bi - Saiku Analytics Founder Tel: +44(0)5603641316 (Thanks

[GitHub] incubator-zeppelin pull request: ZEPPELIN-491: Giving fixed height...

2016-02-08 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/519#issuecomment-181283629 Also I think this should have scroll to bottom of output button. --- If your project is set up for it, you can reply to this email and have your

Re: Zeppelin inside an OSGI container

2016-02-08 Thread Alexander Bezzubov
Hi Tom, it was nice meeting you, my name is Alex and I was a release manager for latest 0.5.6 release. Regarding your questions: - `mvn package -p zeppelin-web` will already build a WAR file for a frontend webapp, that you can deploy to any application container - in order to work with

[GitHub] incubator-zeppelin pull request: Added Kerberos documentation

2016-02-08 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/640#issuecomment-181280614 I left some comments in this file. Except them, LGTM! : ) --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-zeppelin pull request: ZEPPELIN-657: Change the official...

2016-02-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/700#issuecomment-181280722 Thanks. 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

[GitHub] incubator-zeppelin pull request: Add new selenium test case for cr...

2016-02-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/688#issuecomment-181280993 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 on GitHub as well. If

[jira] [Created] (ZEPPELIN-663) Add a SQL Server interpreter to Zeppelin

2016-02-08 Thread Davide Mauri (JIRA)
Davide Mauri created ZEPPELIN-663: - Summary: Add a SQL Server interpreter to Zeppelin Key: ZEPPELIN-663 URL: https://issues.apache.org/jira/browse/ZEPPELIN-663 Project: Zeppelin Issue Type:

[GitHub] incubator-zeppelin pull request: Docs for jdbc interpreter

2016-02-08 Thread vgmartinez
Github user vgmartinez commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/661#issuecomment-181253965 @felixcheung @AhyoungRyu sorry had not see those comments ... I'm a little distracted --- If your project is set up for it, you can reply to

[GitHub] incubator-zeppelin pull request: Add new selenium test case for cr...

2016-02-08 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/688#issuecomment-181284918 @ravicodder I think you should test both append paragraph to top, and append paragraph to bottom buttons as well. Instead of only testing append to

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

2016-02-08 Thread justsomeone1001
Github user justsomeone1001 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-181672499 @bzz Thank you for your attention and efforts to move this forward. I am happy to do the work with respect to the issues you've identified (other

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-662] HBase interpreter ...

2016-02-08 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/701#issuecomment-181706331 @felixcheung How about moving that profile into parent's pom? It looks like that it's used by several poms. --- If your project is set up for it, you can

[GitHub] incubator-zeppelin pull request: ZEPPELIN-646: Shell interpreter o...

2016-02-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/683 --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-491: Giving fixed height...

2016-02-08 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/519#issuecomment-181742675 Have implement follow scroll to the end and scroll to top. If there is a streaming output user can always have an eye on end of scroll.

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-576] Support Apache Dri...

2016-02-08 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-181716616 @tmx11 Hi, could you please those jars under `/interpreters/jdbc/`? I think this would be more general position to use that interpreter. If it still

[GitHub] incubator-zeppelin pull request: Simple improvement to showDF() to...

2016-02-08 Thread markncooper
Github user markncooper closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/699 --- 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

Re: [DISCUSS] Graduate Zeppelin from the Incubator

2016-02-08 Thread moon soo Lee
Hi Greg, On Sat, Feb 6, 2016 at 10:34 PM Greg Stein wrote: > On Thu, Feb 04, 2016 at 10:50:10AM -0800, Mina Lee wrote: > > I think Zeppelin meets the requirements for graduation. > > > > Zeppelin community is growing fast and getting more diverse. Number of > > contributors

[GitHub] incubator-zeppelin pull request: Docs for jdbc interpreter

2016-02-08 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/661#issuecomment-181716399 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

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

2016-02-08 Thread justsomeone1001
Github user justsomeone1001 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-181674445 @bzz Thank you for your attention and efforts to move this forward! I'm happy to do all the required work (but I'm really just stuck on CI).

[GitHub] incubator-zeppelin pull request: selenium test spark, pyspark and ...

2016-02-08 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/654#issuecomment-181732834 Great! If any of these fails is there a simple way to know why? I think recently there was one minor UI change and it was somewhat ambiguous when

[GitHub] incubator-zeppelin pull request: Zeppelin 639

2016-02-08 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/684#issuecomment-181712261 @cjmatta This PR has irrelative commits. Could you please remove them and push your change only? For example, @vgmartinez's commit is reviewed by #667.

[GitHub] incubator-zeppelin pull request: [Zeppelin-628 ] Fix parse propert...

2016-02-08 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/667#issuecomment-181711378 @vgmartinez I think we need to support default interpreter without setting any `default` word. In other words, users should be able to use default jdbc by

[GitHub] incubator-zeppelin pull request: Docs for jdbc interpreter

2016-02-08 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/661#issuecomment-181715598 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

Re: [DISCUSS] Graduate Zeppelin from the Incubator

2016-02-08 Thread moon soo Lee
Amos, As you see, other PPMC started helping your PR. I hope that makes you feel more comfortable. Also I'd like to take your concern about zeppelin's graduation more seriously. So, could you list out your official concern about the zeppelin's graduation? Then me and other PPMC are willing to

[GitHub] incubator-zeppelin pull request: Simple improvement to showDF() to...

2016-02-08 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/699#issuecomment-181715206 @markncooper Could you please close this PR? --- 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: Update NVD3 library for supportin...

2016-02-08 Thread camilosampedro
GitHub user camilosampedro opened a pull request: https://github.com/apache/incubator-zeppelin/pull/704 Update NVD3 library for supporting new features. I have checked it on our cluster and now it makes boxplotcharts. Included features: * Tooltips were refactored

[GitHub] incubator-zeppelin pull request: R and SparkR Support [WIP]

2016-02-08 Thread echarles
Github user echarles commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-181417475 @Leemoonsoo - rscala is now downloaded at build time - bin_license updated - Unit test added - spark-1.3, spark-1.2, spark-1.1 should now

[GitHub] incubator-zeppelin pull request: Add new selenium test case for cr...

2016-02-08 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/688#issuecomment-181357941 CI green, 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

[GitHub] incubator-zeppelin pull request: Add new selenium test case for cr...

2016-02-08 Thread ravicodder
Github user ravicodder commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/688#issuecomment-181356761 @prabhjyotsingh Modified test case to include test case to test append paragraph to top and bottom. Please have a look. --- If your project is set

[GitHub] incubator-zeppelin pull request: R and SparkR Support [WIP]

2016-02-08 Thread justsomeone1001
Github user justsomeone1001 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-181571398 @echarles You could simply have answered him "no, this PR doesn't support R visualizations." I am disinclined to start a thread. I feel

[GitHub] incubator-zeppelin pull request: R and SparkR Support [WIP]

2016-02-08 Thread echarles
Github user echarles commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-181565303 2 failures on last build: spark-1.3:

[GitHub] incubator-zeppelin pull request: R and SparkR Support [WIP]

2016-02-08 Thread smartinsightsfromdata
Github user smartinsightsfromdata commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-181561671 @echarles @justsomeone1001 could you kindly clarify if **native visualization** refers to **Zeppelin visualisations** or R?

[GitHub] incubator-zeppelin pull request: R and SparkR Support [WIP]

2016-02-08 Thread echarles
Github user echarles commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-181568340 @echarles @justsomeone1001 This PR supports ggplot2 and displaying R dataframe in native zeppelin widget ## ggplot2

[GitHub] incubator-zeppelin pull request: Add filter to quickly search the ...

2016-02-08 Thread minahlee
Github user minahlee commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/660#issuecomment-181579567 LGTM merging if there is no more discussion --- 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-595] Allow displaying d...

2016-02-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/685 --- 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

[GitHub] incubator-zeppelin pull request: Docs for jdbc interpreter

2016-02-08 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/661#issuecomment-181606224 @vgmartinez sorry, i think my comment has confused you. Please change this

[GitHub] incubator-zeppelin pull request: ZEPPELIN-657: Change the official...

2016-02-08 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/700#issuecomment-181604900 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

[GitHub] incubator-zeppelin pull request: Add filter to quickly search the ...

2016-02-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/660 --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Add support for us...

2016-02-08 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/697#issuecomment-181636357 Looks like the build failed due to some intermittent NPM errors? Can we re-run these? --- If your project is set up for it, you can reply to this email

[GitHub] incubator-zeppelin pull request: Docs for jdbc interpreter

2016-02-08 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/661#issuecomment-181636680 thanks, merging if no more discussion. --- 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: R Interpreter for Zeppelin

2016-02-08 Thread sourav-mazumder
Github user sourav-mazumder commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-181642378 I would be happy to help on point 3. Regards, Sourav On Mon, Feb 8, 2016 at 3:36 PM, Alexander wrote:

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Add support for us...

2016-02-08 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/697#issuecomment-181642843 @granturing you can re-run CI by closing and reopening the PR --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Add support for us...

2016-02-08 Thread granturing
GitHub user granturing reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/697 ZEPPELIN-656 - Add support for using Azure storage ### What is this PR for? This is to allow the usage of Azure for notebook storage. ### What type of PR is it?

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Add support for us...

2016-02-08 Thread granturing
Github user granturing closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/697 --- 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: Docs for jdbc interpreter

2016-02-08 Thread vgmartinez
Github user vgmartinez commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/661#issuecomment-181620054 Fix --- 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

Re: Concerns about PR208 was Re: [DISCUSS] Graduate Zeppelin from the Incubator

2016-02-08 Thread Alexander Bezzubov
Guys, if somebody could help with first issues from the list in [PR208]( https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-181624753) I'll be happy to help with CI issues over the coming weekend. Please let me know what you guys think - we need to move on all together and enable

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

2016-02-08 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-181624753 Guys, R interpreter is a big important and complicated contribution, hope we could make our last push to get that merged. On my spare time I briefly

[jira] [Created] (ZEPPELIN-664) Add yarn-client support for flink interpreter

2016-02-08 Thread Andrew McLeod (JIRA)
Andrew McLeod created ZEPPELIN-664: -- Summary: Add yarn-client support for flink interpreter Key: ZEPPELIN-664 URL: https://issues.apache.org/jira/browse/ZEPPELIN-664 Project: Zeppelin Issue