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

2016-03-28 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-202727314 @jeffsteinmetz I had a typo in the default template, it should be fixed now and %r show up. Thanks for catching! You are correct that build-distr wo

Re: [DISCUSS] Back to PRs 208 & 702...

2016-03-28 Thread Jeff Steinmetz
I’m not affiliated to either author nor have any attachment to an specific outcome - and happy to continue being as objective and unbiased as possible. I would say they now have different philosophical approaches (as of the March 23rd merge of datalayer#7 to 702). I agree with Amos Elberg tha

[GitHub] incubator-zeppelin pull request: Replace standard basic-auth dialo...

2016-03-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/801#issuecomment-202726499 @prabhjyotsingh Absolutely better than before :+1: --- 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: Fix removing notes without specif...

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

RE: [DISCUSS] Back to PRs 208 & 702...

2016-03-28 Thread Ankur Jain
+1 on merging 208... I am also using 208 repo from long time... Regards, Ankur -Original Message- From: Joel Van Veluwen [mailto:joel.vanvelu...@quantium.com.au] Sent: 29 March 2016 07:21 AM To: dev@zeppelin.incubator.apache.org Subject: RE: [DISCUSS] Back to PRs 208 & 702... +1 We've

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-209] Folder support for...

2016-03-28 Thread felizbear
Github user felizbear commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/796#issuecomment-202717837 Maybe you should do `bower install`? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-209] Folder support for...

2016-03-28 Thread johnnyws
Github user johnnyws commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/796#issuecomment-202716213 It seems some recent change breaks the karma unit tests. I saw such failures while running unit tests: ``` Error: [$injector:modulerr] Failed to in

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

2016-03-28 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-202711880 As a reminder, building the distribution with `-Pbuild-distr` doesn't package the interpreter as expected. --- If your project is set up for it, you c

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

2016-03-28 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-202711574 Should the interpreter list %r in the header? Right now it shows %knitr, but not %r **spark** %spark (default) , %sql , %knitr , %pyspa

[GitHub] incubator-zeppelin pull request: remove overwritten style for p

2016-03-28 Thread johnnyws
Github user johnnyws commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/797#issuecomment-202698867 @felixcheung sorry for the delay. I've uploaded the screenshots --- If your project is set up for it, you can reply to this email and have your reply appear

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

2016-03-28 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-202695901 Things that left to be done in order to merge this PR: - [x] ~~master and this branch have diverged: we need to either merge master in here, or rebase i

[GitHub] incubator-zeppelin pull request: Replace standard basic-auth dialo...

2016-03-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/801#issuecomment-202691934 Thanks for the feedback @Leemoonsoo, have restrict access to notebook(s) without login. --- If your project is set up for it, you can reply to this e

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

2016-03-28 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-202682652 The push earlier tonight *should* (if it doesn't, let me know) resolve all comments I've received about this PR. It currently passes CI for Spark 1.

[GitHub] incubator-zeppelin pull request: ZEPPELIN-748 make websocket maxTe...

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

RE: [DISCUSS] Back to PRs 208 & 702...

2016-03-28 Thread Joel Van Veluwen
+1 We've been super keen for 208 to be merged for some time at my organisation. I think if 702 is really adding value then we can assess it once 208 has been merged. Amos has put a lot of work into this and it delivers a feature that many Zeppelin users have wanted. Can we finally merge and

Re: [DISCUSS] Back to PRs 208 & 702...

2016-03-28 Thread Sourav Mazumder
All said and done we had enough discussion on this point for many months now. As far as I know, 208 is the PR which community/people have so far used mostly and successfully (at least me and whoever I introduced to 208 for SparkR support). I thought it was going to be merged a long time ago. May b

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-770] Add support for Ma...

2016-03-28 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/803#issuecomment-202604280 looks good to me. any more comment? --- 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: [ZEPPELIN-770] Add support for Ma...

2016-03-28 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/803#issuecomment-202604105 test failure was: ``` +tar zxf spark-1.3.1-bin-hadoop2.3.tgz gzip: stdin: not in gzip format tar: Child returned status 1 tar: Err

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-760] Companion object b...

2016-03-28 Thread rawkintrevo
Github user rawkintrevo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/794#issuecomment-202599734 After reviewing the other InterpreterTest files, I find that none utilize a test where the output '.contains' a specific string. That specific test in I

[jira] [Created] (ZEPPELIN-771) GUI issue displaying unicode characters

2016-03-28 Thread Sarah Quigley (JIRA)
Sarah Quigley created ZEPPELIN-771: -- Summary: GUI issue displaying unicode characters Key: ZEPPELIN-771 URL: https://issues.apache.org/jira/browse/ZEPPELIN-771 Project: Zeppelin Issue Type:

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

2016-03-28 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-202590457 This Virtual Machine PR https://github.com/apache/incubator-zeppelin/pull/751 has also been updated to support R and all libraries required for the exa

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

2016-03-28 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-202583441 This still shows `Error` (paragraph header upper right), but doesn't really break anything per se: ``` %r foo <-1 ``` --- If

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

2016-03-28 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-202583079 Should `%r` be added to the interpreter headers? See the following attachments: https://cloud.githubusercontent.com/assets/5488059/14090453/44342

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

2016-03-28 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-202582736 I tested the build and distribution as follows: `mvn clean package -Pspark-1.6 -Ppyspark -Phadoop-2.4 -Psparkr -DskipTests -Pbuild-distr` T

[GitHub] incubator-zeppelin pull request: Enhanced resource pool serializat...

2016-03-28 Thread fireboy1919
Github user fireboy1919 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/782#issuecomment-202581884 The second one. And I have a solution that does that in this pull request now. I didn't remove the interpreter serializer from this pull

Re: Neo4J interpreter - how to exploit json graph results

2016-03-28 Thread Gérard Dupont
Diving into the mail archive, I found this one : https://mail-archives.apache.org/mod_mbox/incubator-zeppelin-dev/201508.mbox/%3CCABePtM0cu4SZSMJLq=steckawup3i_5x9xhjw4fmxtg4s5i...@mail.gmail.com%3E Followed the advice: using HTML interpreter type instead of SVG. It's working (well I still need to

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-770] Add support for Ma...

2016-03-28 Thread cjmatta
Github user cjmatta commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/803#issuecomment-202570853 @felixcheung As far as I know they work without mapr-sepcific libraries. --- If your project is set up for it, you can reply to this email and have your repl

Re: Neo4J interpreter - how to exploit json graph results

2016-03-28 Thread Gérard Dupont
Hi Eric, thx for the prompt answer. much appreciated.Simply said I tried to load a basic SVG sample from a static file as interpreter output, just to see how it behave (code sample http://pastebin.com/vaC0B3M5 ) I got nothing in the notebook (no error, no rendering of the SVG). A new empty paragr

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-770] Add support for Ma...

2016-03-28 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/803#issuecomment-202556152 Would other components like HBase, Hive work with MapR 5.1? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-760] Companion object b...

2016-03-28 Thread rawkintrevo
Github user rawkintrevo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/794#issuecomment-202538580 I moved the test to the end on a hunch that max fix the build issue. @Leemoonsoo can you run again on your machine? I think the code is solid- it just s

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-770] Add support for Ma...

2016-03-28 Thread cjmatta
GitHub user cjmatta opened a pull request: https://github.com/apache/incubator-zeppelin/pull/803 [ZEPPELIN-770] Add support for MapR 5.1 ### What is this PR for? Adding support for MapR 5.1 via profile ### What type of PR is it? Improvement ### Todos

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-760] Companion object b...

2016-03-28 Thread rawkintrevo
GitHub user rawkintrevo reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/794 [ZEPPELIN-760] Companion object bug fix for Flink Interpretter ### What is this PR for? This implements the solution presented for the Spark interpreter in PR-780 for the Flink

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-760] Companion object b...

2016-03-28 Thread rawkintrevo
Github user rawkintrevo closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/794 --- 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 th

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-760] Companion object b...

2016-03-28 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/794#issuecomment-202517074 @rawkintrevo Please feel free to confine this change if it would help. We could follow up separately for Ignite. --- If your project is set up for i

[jira] [Created] (ZEPPELIN-770) Need MapR 5.1 Build profile

2016-03-28 Thread Chris Matta (JIRA)
Chris Matta created ZEPPELIN-770: Summary: Need MapR 5.1 Build profile Key: ZEPPELIN-770 URL: https://issues.apache.org/jira/browse/ZEPPELIN-770 Project: Zeppelin Issue Type: Bug Co

Re: Neo4J interpreter - how to exploit json graph results

2016-03-28 Thread Eric Charles
I would go into SVG what you want to display can be achieved with SVG. Can you share the issue you encounter? On 28/03/16 18:17, Gérard Dupont wrote: Hi guys, I'm currently working (night & week-end not full time) on a neo4J interpreter to try q=and test few things with zeppelin. For context : n

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-183] Interpreter proces...

2016-03-28 Thread EronWright
Github user EronWright commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/795#issuecomment-202470891 How about the options are renamed as follows: - "shared" -> "shared" - "instance" -> "scoped" - "process" -> "isolated" --- If your

Neo4J interpreter - how to exploit json graph results

2016-03-28 Thread Gérard Dupont
Hi guys, I'm currently working (night & week-end not full time) on a neo4J interpreter to try q=and test few things with zeppelin. For context : neo4j is a graph database which is fun to work with. So far, everything good and I managed to get json results in the notebook. Next step would be to int

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-760] Companion object b...

2016-03-28 Thread rawkintrevo
Github user rawkintrevo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/794#issuecomment-202463668 I really don't. I don't know much of anything about Ignite. I really was only doing this for Flink, but Felix asked that I apply the fix to Scalding/Ig

[GitHub] incubator-zeppelin pull request: Enhanced resource pool serializat...

2016-03-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/782#issuecomment-202459832 Thanks for the understanding. I think term 'resource', 'remote resource' are bit confusing in this conversation. if i define, Resource -

[GitHub] incubator-zeppelin pull request: Replace standard basic-auth dialo...

2016-03-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/801#issuecomment-202445244 @prabhjyotsingh Cool improvement! I tried. And found that i can access notebook without login after enables authentication by editing shiro.ini.

[GitHub] incubator-zeppelin pull request: Enhanced resource pool serializat...

2016-03-28 Thread fireboy1919
Github user fireboy1919 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/782#issuecomment-202411857 That makes sense. The issue is that "ResourcePoolUtils" currently doesn't use that enhanced method, and if you're trying to get a specific resource, that

[GitHub] incubator-zeppelin pull request: replace standard alert and confir...

2016-03-28 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/incubator-zeppelin/pull/802 replace standard alert and confirm with BootstrapDialog ### What is this PR for? Replace standard alert and confirm with BootstrapDialog. Most of these were already take ca

[GitHub] incubator-zeppelin pull request: Replace standard basic-auth dialo...

2016-03-28 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/incubator-zeppelin/pull/801 Replace standard basic-auth dialog with BootstrapDialog for user authentication ### What is this PR for? Instead of showing standard basic-auth dialog, replace the same with

[jira] [Created] (ZEPPELIN-769) Replace standard basic-auth dialog with BootstrapDialog for user authentication

2016-03-28 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-769: Summary: Replace standard basic-auth dialog with BootstrapDialog for user authentication Key: ZEPPELIN-769 URL: https://issues.apache.org/jira/browse/ZEPPELIN-769

Re: [DISCUSS] Back to PRs 208 & 702...

2016-03-28 Thread Eran Witkon
@Elberg, If I were you I would ask myself why isn't the community taking part in this debate? Personally I prefer a team player as a contributor over the best developer. just my 2c Eran On Mon, 28 Mar 2016 at 09:52 Amos B. Elberg wrote: > Moon - I opened this discussion so it could take place wi