[GitHub] incubator-zeppelin pull request: ZEPPELIN-783 fix CI failure on Sp...

2016-04-01 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/810#issuecomment-204659044 @felixcheung feedback addressed in 039513b --- 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-772] - Improve text app...

2016-04-01 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/804#issuecomment-204644812 Shouldn't all output be captured? Is there a reason we want to clear it when the user navigates out? Perhaps I'm not fully understanding the behavior cha

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

2016-04-01 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/797#issuecomment-204643388 @johnnyws what's your JIRA account name? Would like to assign this fix to you. Thanks --- If your project is set up for it, you can reply to this email

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-772] - Improve text app...

2016-04-01 Thread echarles
Github user echarles commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/804#issuecomment-204642561 The expected behavior I would expect is the "Before" one. If I ask for a result, it should be consistent, whatever page I navigate in between. I wou

[GitHub] incubator-zeppelin pull request: Add R Dependencies to Virtual Mac...

2016-04-01 Thread echarles
Github user echarles commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/751#issuecomment-204641943 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: R and SparkR Support [WIP]

2016-04-01 Thread echarles
Github user echarles commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-204641032 Back to green after simple build retrigger. LGTM for merge. --- If your project is set up for it, you can reply to this email and have your reply a

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

2016-04-01 Thread asfgit
Github user asfgit 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 the fea

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

2016-04-01 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-204616417 @bzz are we good? --- 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-04-01 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-204612256 We may be good now - the only failures on the last push were of the CI-random type. --- If your project is set up for it, you can reply to this email and ha

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

2016-04-01 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-204597130 Yeah, I may let it load from the Zeppelin R library if we can't find another solution. I really don't want to use a user-specific directory, because

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

2016-04-01 Thread smartinsightsfromdata
Github user smartinsightsfromdata commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-204586709 @elbamos A minor comment: `sudo` type of solution won't work on windows, of course. From a R perspective, the alternatives I can se

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-707]Automatically adds ...

2016-04-01 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-204586660 Does this work when the user has not included `%interpreter` in the previous paragraph? (ie. using the default interpreter) --- If your project is set

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-781] Update Flink inter...

2016-04-01 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/809#issuecomment-204584598 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: remove overwritten style for p

2016-04-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/797 --- 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-04-01 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-204582739 @bzz Here's where we are: * I believe all the licensing, rat, and rebasing to-do's are resolved. * On CI, trying to follow your proposal, if we turn-o

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

2016-04-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/799 --- 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: [ZEPPELIN-768] HBase 1.1.x and 1....

2016-04-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/800 --- 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: ZEPPELIN-783 fix CI failure on Sp...

2016-04-01 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/810#issuecomment-204571013 Right, dependencies download is another frequent source of download timeout issues. Added a comment on that. --- If your project is set up for it, you

Re: Spark minimum version support

2016-04-01 Thread Amos Elberg
+1 I can't think of any reason why anyone using a version of Spark before 1.3 would use Zeppelin. And versions before 1.4 are pretty unlikely as well. Getting rid of 1.1 and 1.2 - and even 1.3 - would make everyone's lives easier. Spark 2.0 is coming out soon, so if we can clear out some of tho

Spark minimum version support

2016-04-01 Thread Jeff Steinmetz
Curious if there is there has been any discussion regarding the minimum version of Spark to support in Zeppelin. For example – it could be “current version plus 2 previous versions”. 1.6.x, 1.5.x, 1.4.x (if Spark ever changes the major version to 2.x.x, it could be latest Major version and pr

Re: Trigger Travis build (was: Re: [DISCUSS] Back to PRs 208 & 702...)

2016-04-01 Thread Amos B. Elberg
When I force push, I insert white space into a file where it will be ignored to trick git. > On Apr 1, 2016, at 12:26 PM, Eric Charles wrote: > > On 30/03/16 23:04, DuyHai Doan wrote: > > (snip...) > >> --> Same problem here, some times some of my PRs are just red because of >> random failur

Trigger Travis build (was: Re: [DISCUSS] Back to PRs 208 & 702...)

2016-04-01 Thread Eric Charles
On 30/03/16 23:04, DuyHai Doan wrote: (snip...) --> Same problem here, some times some of my PRs are just red because of random failure, the only way to make it green is to *repeatedly* force-push until it passes green. If there is no change, my git tells me there is nothing to push git pus

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

2016-04-01 Thread fireboy1919
Github user fireboy1919 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/782#issuecomment-204451849 @Leemoonsoo: does this address everything? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] incubator-zeppelin pull request: ZEPPELIN-783 fix CI failure on Sp...

2016-04-01 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/810#issuecomment-204424456 @echarles So true. I plan to take care of that as a next step under a separate PR, after dealing with Spark downloads here first. --- If your project is set up

[GitHub] incubator-zeppelin pull request: ZEPPELIN-783 fix CI failure on Sp...

2016-04-01 Thread echarles
Github user echarles commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/810#issuecomment-204409082 That's a nice improvement which will remove some failures, but download failures also occur on simple jars (writing this down while thinking to those errati

Running zeppelin webapp within a tomcat server

2016-04-01 Thread Minudika Malshan
Hi all, I want to integrate zeppelin as a webapp within another server which is running on top of tomcat server. By adding just zeppelin's .war file to the webapp folder it deploys it as an web app and I am able to access to its index file. But it's not functional. How to get it properly function

[GitHub] incubator-zeppelin pull request: ZEPPELIN-783 fix CI failure on Sp...

2016-04-01 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/810#issuecomment-204388627 Ready to be merged \cc @felixcheung @jongyoul @Leemoonsoo for a review --- If your project is set up for it, you can reply to this email and have your reply appe

RE: Issue with SparkSqlInterpreter

2016-04-01 Thread Ghosh, Parthapratim
ZEPPELIN-785 has been raised for the below mentioned issue. Thanks and Regards Partha Pratim Ghosh From: Ghosh, Parthapratim [CCC-OT NE] Sent: Friday, April 01, 2016 8:37 AM To: 'us...@zeppelin.incubator.apache.org' Cc: 'dev@zeppelin.incubator.apache.org' Subject: Issue with SparkSqlInterpreter

[jira] [Created] (ZEPPELIN-785) Issue with SparkSqlInterpreter

2016-04-01 Thread Partha Pratim Ghosh (JIRA)
Partha Pratim Ghosh created ZEPPELIN-785: Summary: Issue with SparkSqlInterpreter Key: ZEPPELIN-785 URL: https://issues.apache.org/jira/browse/ZEPPELIN-785 Project: Zeppelin Issue Typ

Issue with SparkSqlInterpreter

2016-04-01 Thread Ghosh, Parthapratim
Hi, I am having an issue with %sql interpreter in Zeppelin. I am running Zeppelin 0.5.6. Each time a query is made in %sql (SparkSqlInterpreter), the interpreter needs to be restarted for the next query to work. If we try to run two select queries

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

2016-04-01 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-204351155 @ankurmitujjain please refer https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-203834473 for things that need to be taken care of before merging

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

2016-04-01 Thread echarles
Github user echarles commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-204327315 Thx @prabhjyotsingh for the hint. I have merged and there is no more conflict. Travis failed for 1.1.1 profile (issue while downloading a jar), othe

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-772] - Improve text app...

2016-04-01 Thread vgmartinez
Github user vgmartinez commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/804#issuecomment-204325291 I testedLGTM --- 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

RangeError: Maximum call stack size exceeded

2016-04-01 Thread 강민우
Hi. I'm interesting in contributing code to Zeppelin.(I am not good in English. Sorry for mistakes.) So I'm making development environment in my computer.But I had a problem. I want to your help. I tried to run './grunt serve' in '~/incubator-zeppelin/zeppelin-web' folder after I run 'mvn clean

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

2016-04-01 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-204289348 @echarles rebase with master. --- 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