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

2016-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/734 --- 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-647] - Native Windows s...

2016-03-23 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-200441262 LGTM. 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 wel

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

2016-03-21 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-199595583 Most of the profiles are green, including those with tests, so LGTM --- If your project is set up for it, you can reply to this email and have your reply

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

2016-03-21 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-199591162 One version failed because it couldn't download Spark, the other because of connection refused errors running API tests. My last commit was only against t

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

2016-03-21 Thread granturing
Github user granturing closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/734 --- 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-647] - Native Windows s...

2016-03-21 Thread granturing
GitHub user granturing reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/734 [ZEPPELIN-647] - Native Windows support for startup scripts and configuration ### What is this PR for? This is to give Windows first-class support for running Zeppelin without

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

2016-03-21 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-199579294 @granturing Selenium tests failed, could you close/re-open this PR to launch CI again? --- If your project is set up for it, you can reply to this email

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

2016-03-19 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-198037337 I have no more changes. @dnldxn were you able to validate the latest commit on your setup? --- If your project is set up for it, you can reply to this em

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

2016-03-19 Thread dnldxn
Github user dnldxn commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-198226908 @granturing Yep, it works great on my Windows 10 system. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

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

2016-03-19 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-198036050 Is it okay to be merged? --- 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

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

2016-03-18 Thread Amos B. Elberg
This PR needs to be a broader discussion. If Zeppelin is going to support windows, that means making a commitment to support windows. That's means supporting a wider variety of configurations, and varying levels of user ability, orders of magnitude beyond what we do now. How are we going to CI

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

2016-03-14 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-196634863 @dnldxn thanks for testing that. I changed a few things to better handle spaces. Unfortunately, using the bundled Spark with a ZEPPELIN_HOME with spaces w

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

2016-03-14 Thread dnldxn
Github user dnldxn commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-196556262 The script seems to have a problem with CLASSPATH's that contain spaces (e.g. C:\Program Files\xyz\bin\). I'm not knowledgeable enough with with Windows Batc

[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: [ZEPPELIN-647] - Native Windows s...

2016-03-09 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-194682205 I've been running this extensively the past few weeks and have nothing left to add at this point, unless there's additional feedback. --- If your project

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

2016-03-05 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-192776037 Couple questions: * The Shell interpreter uses bash, which obviously won't work on Windows. I've confirmed the interpreter will work with a couple

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

2016-03-03 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-192086401 Hmm, interesting. Let's keep this here for now then. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

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

2016-03-03 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-191828541 So, interestingly enough I only see an issue with Spark (using Hadoop 2.6). Running Flink seems fine, but it could be dependent on the Hadoop client versi

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

2016-03-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-191472507 +1 for track the Windows Cassandra test failure in a new JIRA --- 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-647] - Native Windows s...

2016-02-28 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-189935705 Hi Felix, agree with the Cassandra test failure. I have a couple more things I want to verify and then we should be good to go. I'm writing up some docs r

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

2016-02-28 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-189935535 The last CI failure is because of spark release download timeout. I think we are good to go - how about we track the Windows Cassandra test failure in

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

2016-02-28 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-189873259 Let me checkout the project on Windows and try (I need to find a Windows :D) --- If your project is set up for it, you can reply to this email and have

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

2016-02-28 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-189872676 @felixcheung Yes, **Cassandra** is supported on Windows. The error is about parsing multi line comments. The first test failure looks in

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

2016-02-27 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-189785823 @doanduyhai is cassandra supported on windows? --- 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: [ZEPPELIN-647] - Native Windows s...

2016-02-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-187097842 I tried it on Windows 7, works well. Except for these two test failure in cassandra, when I do `SPARK_VER="1.6.0" HADOOP_VER="2.3" PROFILE

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

2016-02-21 Thread ankurmitujjain
Github user ankurmitujjain commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-187021997 It works on Windows 7 as well. --- 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

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

2016-02-20 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-186749794 It works well on my windows XP with oracle jdk 1.7.0_79. great work! --- If your project is set up for it, you can reply to this email and have your reply

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

2016-02-20 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-186653407 @granturing Appreciate for the patch! I'll manually test it in this evening with my windows XP and share the result. --- If your project is set up f

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

2016-02-20 Thread granturing
GitHub user granturing opened a pull request: https://github.com/apache/incubator-zeppelin/pull/734 [ZEPPELIN-647] - Native Windows support for startup scripts and configuration ### What is this PR for? This is to give Windows first-class support for running Zeppelin without th