[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-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 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 #1534: ZEPPELIN-1345 - Create a custom matplotlib backend tha...

2016-11-06 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1534 LGTM. Is there any known issue with SELENIUM tests? from a quick glance it seems to have failed a few times here. --- If your project is set up for it, you can reply to this email and hav

[GitHub] zeppelin pull request #1606: [ZEPPELIN-777] Math formula support

2016-11-06 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/1606 [ZEPPELIN-777] Math formula support ### What is this PR for? This PR adds support for formatting math formula formatting in %html display system using MathJax library. Note that,

[GitHub] zeppelin issue #1534: ZEPPELIN-1345 - Create a custom matplotlib backend tha...

2016-11-06 Thread agoodm
Github user agoodm commented on the issue: https://github.com/apache/zeppelin/pull/1534 There is a JIRA issue: ([ZEPPELIN-1623](https://issues.apache.org/jira/browse/ZEPPELIN-1623)) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #1606: [ZEPPELIN-777] Math formula support

2016-11-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1606 cc @1ambda @AhyoungRyu --- 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 wi

[GitHub] zeppelin issue #1594: [ZEPPELIN-1614] Remove markdown4j dep

2016-11-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1594 Thanks @1ambda for the contribution. I have compared output from markdown4j and pegdown and found some differences. font difference ![image](https://cloud.githubusercontent.com

[GitHub] zeppelin issue #1594: [ZEPPELIN-1614] Remove markdown4j dep

2016-11-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1594 And i think it'll be better mention in `docs/install/upgrade.html#upgrading-from-zeppelin-06-to-07` about default markdown parser change and result slightly different output rendering of markd

[GitHub] zeppelin issue #1596: ZEPPELIN-1616. Interpreter open happens in jobRun

2016-11-06 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1596 @agoodm @Leemoonsoo I found the issue, this is due that to `PySparkInterpreter` depends on `SparkInterpreter`. So before opening `PySparkInterpreter` we should creating `SparkInterpreter`. But see

[GitHub] zeppelin issue #1596: ZEPPELIN-1616. Interpreter open happens in jobRun

2016-11-06 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1596 Think about it more, I don't have strong preference on this PR. As name `LazyOpenInterpreter` seems to imply it would call `open` method when `interpret` method is called. My purpose is to try to p

[GitHub] zeppelin issue #1606: [ZEPPELIN-777] Math formula support

2016-11-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1606 @AhyoungRyu @anthonycorbacho addressed your comments. --- 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] zeppelin pull request #1433: [ZEPPELIN-1451] Bug fix of Embedding %html with...

2016-11-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1433 --- 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 feature is e

[GitHub] zeppelin issue #1594: [ZEPPELIN-1614] Remove markdown4j dep

2016-11-06 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1594 @Leemoonsoo Thanks for review 👍 It would be better to 1. Use pegdown as default 2. Add docs for it 3. Revert markdown4j Regarding to rendered output you provide

[GitHub] zeppelin issue #1596: ZEPPELIN-1616. Interpreter open happens in jobRun

2016-11-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1596 I think cleaner way to remove `open()` inside of `jobRun()` could be call `open()` inside of [LazyOpenInterpreter.getHook()](https://github.com/apache/zeppelin/blob/master/zeppelin-interpreter/

[GitHub] zeppelin issue #1538: [ZEPPELIN-1386] Docker images for running Apache Zeppe...

2016-11-06 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1538 @mfelgamal Could you tell me more detail about the issue on `from 0.6.0v to 0.6.2v` ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] zeppelin issue #1534: ZEPPELIN-1345 - Create a custom matplotlib backend tha...

2016-11-06 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1534 Great, thank you for digging it through. Any more comment? --- 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] zeppelin issue #1538: [ZEPPELIN-1386] Docker images for running Apache Zeppe...

2016-11-06 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1538 @mfelgamal I tried `zeppelin-release:0.6.0` image but it failed to run spark code in `Zeppelin Tutorial` ``` $ docker ps CONTAINER IDIMAGE

[GitHub] zeppelin issue #1538: [ZEPPELIN-1386] Docker images for running Apache Zeppe...

2016-11-06 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1538 Additionally, It would be better to - run zeppelin immediately (using `zeppelin.sh` because we can run docker as daemon mode) - allow user to modify config important config files by pa

[GitHub] zeppelin issue #1588: [ZEPPELIN-1610] [WIP] - Add notebook watcher

2016-11-06 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1588 @anthonycorbacho thanks for contribution. as far as i can see, this is kind of promiscuous mode for websockets. @Leemoonsoo any comments? Also maybe remove `WIP` from title --- If your pr