[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-11-07 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/928 Looks great to me, thank you @rawkintrevo CI failure looks not relevant to the changes. Merging to master if there is no further discussion. --- If your project is set up for it, you

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-11-07 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 Profile 2 failed. ``` 16/11/06 14:18:51 INFO PySparkInterpreter: File /tmp/zeppelin_pyspark-1526538107058607857.py created Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Tim

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-11-06 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 @felixcheung thanks. Apache Mahout is a kind of special case, in that really we're a set of libraries that can run on currently Spark, Flink, H20 runners, and possible additionals ones (such a

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-11-06 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 @bzz, - I could see usecases either way. So i've added the `--overwrite_existing` flag. If that flag is called it will delete current `%sparkMahout` and `%flinkMahout` interpreters.

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-11-06 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/928 @rawkintrevo thank you for addressing the feedback promptly. I tried examples and everything worked very well! One thing though - on the second run of `add_mahout.py` I got ```

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-11-04 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/928 Interesting approach! --- 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 wis

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-11-04 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 - [x] Fail fasts: changed `call` to [`check_call`](https://docs.python.org/2/library/subprocess.html#check_call) where it is appropriate to fail and exit. - [x] Slang in standard out: Upda

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-11-04 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 I can address the 3 bullet points in the next day or two. Re; the disk space- yea Mahout adds another 300MB or so. I recall seeing some chat on the mailing list that Zeppelin was alread

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-11-04 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/928 Sorry for delay, @rawkintrevo ! Thank you for kind reminder on this great stuff. I want to mention this work on ApacheCon, so hope we can merge this asap :) I tied it and got `IOError

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-10-19 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 bump- status or thoughts 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 project does not have this feature enab

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-10-07 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 Hey @bzz, did you get a chance to check this out? Anything I can do to help? --- 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] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-10-02 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/928 Sounds awesome, @rawkintrevo, thank you for keeping up a good work! Let me try it this week and get back to you. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-09-28 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 Refactored as previously discussed. Added script that sets up two new terps, one for mahout on flink, one for mahout on spark. Added docs and a robust tutorial that shows useage. --- If you

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-09-07 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 @bzz and @Leemoonsoo - Was talking to some other about this- I think we should take a new tack on this as Mahout currently only supports Spark v1.5 officially, but also 1.6 in practice

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-09-07 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 *bump* @bzz should we continue work on this or pursue another route? --- 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] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-22 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 for what it's worth (probably not much) this passed travis CI on my branch- [https://travis-ci.org/rawkintrevo/incubator-zeppelin/builds/154183905](https://travis-ci.org/rawkintrevo/incubat

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-09 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 @bzz I was really hoping to get it in by 0.6.1 but it's whatever. [Profile 1](https://s3.amazonaws.com/archive.travis-ci.org/jobs/150945703/log.txt) is the only failure now- `Par

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-03 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/928 @rawkintrevo you did titanic job, struggling with CI here.. but now, after Spark 2.0 and Scala 2.11 support got merged CI fails even on *master* Would it make sense to take a break until 0.6.1

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-02 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 [Build 6281](https://travis-ci.org/apache/zeppelin/builds/149199571) Profiles [1](https://s3.amazonaws.com/archive.travis-ci.org/jobs/149199572/log.txt), [2](https://s3.amazonaws.com/a

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-08-02 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 ## CI Failures in This PR Profile [2](https://s3.amazonaws.com/archive.travis-ci.org/jobs/148308638/log.txt) Hangs a la [ZEPPELIN-862](https://issues.apache.org/jira/browse/ZEPPE

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-07-28 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 @bzz only difference between this and last was squash of commit. all fails are related to downloading dependencies still. --- If your project is set up for it, you can reply to this email an

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-07-28 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 @bzz and @jongyoul OK, so two things I think have happened. (In case some thing like this happens again). When I rebased, some version of ZeppelinClusterTest.java that had the [

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-07-27 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 I resubmitted this a few times, there is something going awry with finding `org.apache.apache.resources:apache-jar-resource-bundle:jar:1.5-SNAPSHOT` The build fails on random profi