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

2016-03-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/761#issuecomment-195216242 I see following issues: - There is a horizontal scrollbar - When i scroll, table data goes above header https://cloud.githubuser

[GitHub] incubator-zeppelin pull request: Hot fix grunt failure

2016-03-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/772 --- 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: Hot fix grunt failure

2016-03-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/772#issuecomment-195203194 right, CI passed clean as of the last run from yesterday. --- 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: R and SparkR Support [WIP]

2016-03-10 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-195196436 A few tests results: this shouldn't return an error ``` %r foo <- 1 ``` expected `> foo <- 1` also

[GitHub] incubator-zeppelin pull request: Hot fix grunt failure

2016-03-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/772#issuecomment-195144733 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: Hot fix grunt failure

2016-03-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/772#issuecomment-195128515 @r-kamath Could you re-trigger the CI to be sure? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

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

2016-03-10 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195114026 Yes you did understand - I just checked and I know why it didn't work for you, I'll fix it next time I'm in the code. The xtable package generates

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

2016-03-10 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195084517 To confirm, are you saying this should return Zeppelin displayed html? (I may have misunderstood) ``` %r cat("%html hello!”)

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

2016-03-10 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195077353 `print(mtcars) ` returns text in a similar way as R as you suggested. Perhaps some hints in the interpreter docs. Not seeing the ro

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

2016-03-10 Thread justsomeone1001
Github user justsomeone1001 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195074172 I just want to reiterate, I am open on these issues. But who I think we should look to hear from regarding user expectations, are people who are regu

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

2016-03-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/763#issuecomment-195058858 Please 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 your project does not ha

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

2016-03-10 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195057406 The row id's aren't part of the table in R. Also, R actually is formatting the table in a specific way. Try returning a data_frame vs a data.frame in R and s

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

2016-03-10 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195055673 Many R users prefer to use knitr over a repl. Knitr also enables a lot of things that would otherwise be very complex, like packages for producing properly f

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

2016-03-10 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195054465 The R coding style standard is that variables defined by a package are prefixed with a '.' It seemed to me that "z" stands a good chance of being overwritten

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

2016-03-10 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195053667 In R, when a string gets returned to the repl by an expression, it gets formatted by calling print(). This is why the magic isn't working as you expect-R is

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-688] Giving an option t...

2016-03-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/764 --- 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: R Interpreter for Zeppelin

2016-03-10 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195046726 curious why you chose to start the `.z.show` method calls with a period? why not `z.show` --- If your project is set up for it, you can reply

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

2016-03-10 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195050282 Does anybody have feedback regarding the use of `%knitr` for plotting? Seems having to use the `%knitr` interpreter for certain workflows is unneces

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

2016-03-10 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195051151 RE: the `mtcars` example above, R and rstudio returns the dataframe as simple text, not a formatted table and with all of the correct row ids.

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

2016-03-10 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195050132 The standard in R is that the return value of an expression is determined by the interpreter. When a data.frame is returned, the interpreter in the R distro

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

2016-03-10 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195045409 This doesn't work as I would expect: ``` %r print("%html HTML Works") ``` and %table, %md, etc Shouldn't this work similar t

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

2016-03-10 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195041973 It can be changed. I like verbose logging to help diagnose user issues. Logging especially the spark contection is very beneficial. But the level of logging

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

2016-03-10 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195041323 I don't think this should output the result as a zeppelin table: ``` %r mtcars ``` Since it's not in line with how R or R S

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

2016-03-10 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-195039526 I noticed some initial verbose logging - ``` Package Startup Message:

Re: Travis Build flags for tests

2016-03-10 Thread Felix Cheung
In order to minimize duplication, only the first in Travis matrix is running every tests (excluding selenium, which is separate) On Thu, Mar 10, 2016 at 1:56 AM -0800, "Corneau Damien" wrote: I just looked at this PR ( https://github.com/apache/incubator-zeppelin/pull/772) which has C

[GitHub] incubator-zeppelin pull request: Hot fix grunt failure

2016-03-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/772#issuecomment-195013640 earlier test failure was: ``` [31m- should provide onclick method *** FAILED ***  The code passed to eventually never returned normal

[jira] [Created] (ZEPPELIN-734) pyspark importing multiprocessing error

2016-03-10 Thread Maziyar Boustani (JIRA)
Maziyar Boustani created ZEPPELIN-734: - Summary: pyspark importing multiprocessing error Key: ZEPPELIN-734 URL: https://issues.apache.org/jira/browse/ZEPPELIN-734 Project: Zeppelin Issue

[GitHub] incubator-zeppelin pull request: Fixed some documentation

2016-03-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/773#issuecomment-194962672 Thanks for the fix. 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 d

[GitHub] incubator-zeppelin pull request: Fixed some documentation

2016-03-10 Thread michrawson
GitHub user michrawson opened a pull request: https://github.com/apache/incubator-zeppelin/pull/773 Fixed some documentation ### What is this PR for? Fixed some documentation ### What type of PR is it? Documentation You can merge this pull request into a Git r

[ZEPPELIN-683] GSOC 2016 - Darien Yeung

2016-03-10 Thread Darien Yeung
Dear Mr. Bezzubov. Hello my name is Darien Yeung and I am a 4th Year Computer Science and Chemistry student at Vancouver Island University in Canada. I am interested participating in the Google Summer of Code 2016 at Apache and NF Labs and I would like to work on the Project (ZEPPELIN-683):

Re: R interpreter in Zeppelin: further steps

2016-03-10 Thread Amos B. Elberg
Alex - what is the eta on the work on ci for 208? This has been (supposedly) preventing merge since September, and we haven't seen any progress at all. Fixing ci does not need to wait on changes to address the recent re-engineering of the Interpreter architecture. 208 should not be forced to

Re: [Gsoc - 2016][Zeppelin - 683]

2016-03-10 Thread Alexander Bezzubov
Hi Onkar, great to hear the you are interested and thank you for sharing the example notebook that you'v built, preview [0] looks great. I encourage you review this mailing list archives very carefully, looking for the advices to other students on how to get started with zeppelin and proceed with

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

2016-03-10 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/740#issuecomment-194830614 @Leemoonsoo > Understood. Sounds nice. (what happen i import the same note again?) You can re-importe the same note many times, the note

Fwd: Google Summer of Code 2016 Mentor Registration

2016-03-10 Thread Alexander Bezzubov
Dear committers and fellow PPMC members, it's time for prospect mentors to register for GSoC 2016. Please follow the instructions below and let me know if you need any assistance. -- Alex -- Forwarded message -- From: Ulrich Stärk Date: Wed, Mar 9, 2016 at 2:01 AM Subject: Googl

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

2016-03-10 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/740#issuecomment-194821919 Yes @corneadoug because: 1. binding angular object at note level will require binding this value to **all** interpreters enabled in this note. Thi

Re: R interpreter in Zeppelin: further steps

2016-03-10 Thread Alexander Bezzubov
Thank you everybody for expressing valuable opinions: Enzo, Amos, Jeff, Eran, Damien, Eric, Sourav, Luciano, DuyHai, Moon - I'm very glad to see the misunderstanding being finally resolved. The solely purpose of this thread was to _document the community consensus_ and please correct me if I'm wro

[jira] [Created] (ZEPPELIN-733) SparkSqlInterpreter fails to execute query with spark-csv

2016-03-10 Thread JIRA
Raphaël Luta created ZEPPELIN-733: - Summary: SparkSqlInterpreter fails to execute query with spark-csv Key: ZEPPELIN-733 URL: https://issues.apache.org/jira/browse/ZEPPELIN-733 Project: Zeppelin

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-647] - Native Windows s...

2016-03-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-194787522 @granturing Thanks for great work. LGTM. Can https://github.com/apache/incubator-zeppelin/pull/749 and https://github.com/apache/incubator-zeppelin/pull/7

[GitHub] incubator-zeppelin pull request: Option to restart interpreter fro...

2016-03-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/480#issuecomment-194783809 Sorry, I had a pile of notifications. It depends on how the error is detected, the best could be to fire the modal on error (if multiple errors, th

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

2016-03-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/740#issuecomment-194776078 Just a quick question in case I missed something, why are we limiting the angularBind to paragraph level only? --- If your project is set up for it, you

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

2016-03-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/761#issuecomment-194768156 Thanks, I'll test that out --- 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

[GitHub] incubator-zeppelin pull request: Hot fix grunt failure

2016-03-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/772#issuecomment-194766282 looks like the failure is not related to this change 31m*** 1 TEST FAILED *** [INFO] ---

[GitHub] incubator-zeppelin pull request: Hot fix grunt failure

2016-03-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/772#issuecomment-194763990 LGTM, I'm trying to figure out why the CI is failing --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

Re: Travis Build flags for tests

2016-03-10 Thread Corneau Damien
I just looked at this PR ( https://github.com/apache/incubator-zeppelin/pull/772) which has CI failing (1.6 profile only) And after comparing 1.6 to 1.5, the logs are showing that 1.5 are skipping tests while 1.6 doesn't (at least in Display systems api) On Thu, Mar 10, 2016 at 2:22 AM, Felix Cheu

Re: ZEPPELIN-682

2016-03-10 Thread Minudika Malshan
Hi Alex, As we know the origin of Apache beam is the Google cloud platform. It has been developed mainly focusing on deploying pipelines in the cloud. But AFAIK mostly zeppelin is used as a standalone system. Is it expected to implement the cloud support for the interpreter under this GSoC projec

Re: Google Summer of Code 2016 - Darien Yeung

2016-03-10 Thread Darien Yeung
Dear Mr. Bezzubov, Hello, how are you? I had closer look at the Zeppelin notebook json files and I have question I would like to ask you. How does the Notebook json reference which database server to query? I was confused in that aspect as it would important to reference a remote DB

[GitHub] incubator-zeppelin pull request: Hot fix grunt failure

2016-03-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/772#issuecomment-194745698 @corneadoug good find. fixed in 23638b6. thanks --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we