[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

[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

[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-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

[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

[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

[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

[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

[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

[GitHub] incubator-zeppelin pull request: Bringing compiledScope for backwa...

2016-02-19 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/720#issuecomment-186340436 The notebook is code I wrote a while ago in order to show an example of "reactive" client-side JavaScript charts. Is there some other w

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Add support for us...

2016-02-12 Thread granturing
GitHub user granturing reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/697 ZEPPELIN-656 - Add support for using Azure storage ### What is this PR for? This is to allow the usage of Azure for notebook storage. ### What type of PR

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Add support for us...

2016-02-12 Thread granturing
Github user granturing closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/697 --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Add support for us...

2016-02-08 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/697#issuecomment-181462386 Please see these latest commits. First, is to make changes as previously discussed for documenting optional "user" parameter as well as properly

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Add support for us...

2016-02-08 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/697#issuecomment-181636357 Looks like the build failed due to some intermittent NPM errors? Can we re-run these? --- If your project is set up for it, you can reply to this email

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Add support for us...

2016-02-08 Thread granturing
GitHub user granturing reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/697 ZEPPELIN-656 - Add support for using Azure storage ### What is this PR for? This is to allow the usage of Azure for notebook storage. ### What type of PR

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Add support for us...

2016-02-08 Thread granturing
Github user granturing closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/697 --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Add support for us...

2016-02-05 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/697#issuecomment-180645409 OK, I went ahead and added the license info for the Azure storage library as well as the docs. The S3 repo has a "user" folder setting, so

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Support saving not...

2016-02-04 Thread granturing
GitHub user granturing opened a pull request: https://github.com/apache/incubator-zeppelin/pull/696 ZEPPELIN-656 - Support saving notebooks to Azure cloud storage ### What is this PR for? This is to allow the usage of Azure for notebook storage. ### What type of PR

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Support saving not...

2016-02-04 Thread granturing
Github user granturing commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/696#issuecomment-180173476 Apologies for this, opened a new PR with proper commits --- 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-656 - Add support for us...

2016-02-04 Thread granturing
GitHub user granturing opened a pull request: https://github.com/apache/incubator-zeppelin/pull/697 ZEPPELIN-656 - Add support for using Azure storage ### What is this PR for? This is to allow the usage of Azure for notebook storage. ### What type of PR

[GitHub] incubator-zeppelin pull request: ZEPPELIN-656 - Support saving not...

2016-02-04 Thread granturing
Github user granturing closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/696 --- 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