[GitHub] incubator-zeppelin pull request: [ZEPPELIN-672] Add feature to exp...

2016-03-13 Thread ankurmitujjain
Github user ankurmitujjain commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/761#issuecomment-196153440 Thanks for reviewing @prabhjyotsingh -I will look into horizontal scroll bar -However fixed header, is the data tables feature I added, so

Re: R Interpreter - PR 702

2016-03-13 Thread Jeff Steinmetz
I view these comments are more general (not just for PR702). Mainly thoughts around package management globally vs per user. For example, users will need to install `evaluate` and `knitr` to ensure PR208 works (part of setup prior to zeppelin, potentially globally as root). I’ve testeded 208, and

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-713] Expand ResourceId ...

2016-03-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/763#issuecomment-196085361 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 y

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-689] Add AngularJS z ob...

2016-03-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/740#issuecomment-196081077 @doanduyhai Could you take a look a comment for `pushAngularObjectRegistryToRemote(client);` inside of RemoteInterpreter.init() --- If your project is

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-735] Remove spark.execu...

2016-03-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/774#issuecomment-196080473 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

Re: R Interpreter - PR 702

2016-03-13 Thread Amos B. Elberg
@enzo - 208 does not need an R administrator in server of single user mode. This is because the R package that 208 uses is segregated--the same approach is used by other server-based R systems like RStudio and shiny. 702, the problem is unsolvable except by locking-down the system's R installat

Re: R Interpreter - PR 702

2016-03-13 Thread enzo
Just a side consideration on package management, expanding some of Jaff’s comments. As well all know R is in essence a “single user” tool. I assume at the beginning Zeppelin will be the same. On my machine I have two libraries where my R packages are stored, a system library and a user librar

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-666] Change location of...

2016-03-13 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/766#issuecomment-196050734 Thanks @Leemoonsoo for the review. --- 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-699] Return Paragraph e...

2016-03-13 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-196040202 @felixcheung I'm still waiting for someone to reply me on my question: > By the way, for the existing asynchronous API, I don't think returning OK

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-699] Return Paragraph e...

2016-03-13 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-196036497 hello - where are we on this? --- 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 pro

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-157] Adding Map Visuali...

2016-03-13 Thread Madhuka
Github user Madhuka commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/765#issuecomment-196011931 1. You have do it from script for now. (more thing we can add later as you explain) 2. It is not about embeds a leaflet div into a paragraph.[This]

[jira] [Created] (ZEPPELIN-746) Saving notebooks in S3 - allow instance profile credentials provider

2016-03-13 Thread JIRA
Florian Troßbach created ZEPPELIN-746: - Summary: Saving notebooks in S3 - allow instance profile credentials provider Key: ZEPPELIN-746 URL: https://issues.apache.org/jira/browse/ZEPPELIN-746 Proj