[GitHub] incubator-zeppelin pull request: ZEPPELIN-490 Tidying up dynamicfo...

2015-12-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/518#issuecomment-163560443 Working well. Looks good to me --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] incubator-zeppelin pull request: ZEPPELIN-496 ] Bug fixed Spark sc...

2015-12-10 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/528 --- 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

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-495] Enable running int...

2015-12-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/524#issuecomment-16346 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 well. If

[GitHub] incubator-zeppelin pull request: ZEPPELIN-305 Do not add jvm memor...

2015-12-10 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/526#issuecomment-163569976 Looks good to me --- 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

[GitHub] incubator-zeppelin pull request: Add job operation API

2015-12-10 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/465#issuecomment-163572633 Looks awesome to me! Merging if there is no further discussion. --- 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-484] A small utility to...

2015-12-10 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/513#issuecomment-163574301 Looks great to me --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-305 Do not add jvm memor...

2015-12-10 Thread minahlee
Github user minahlee commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/526#issuecomment-163819332 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: Improve/split paragraph html

2015-12-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/324#issuecomment-163826291 \o/ --- 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: Improve/split paragraph html

2015-12-10 Thread corneadoug
GitHub user corneadoug reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/324 Improve/split paragraph html Paragraph has been for long time a difficult part of Zeppelin-web code base. Both Html and JS are quite big, and can be hard to understand. This

[GitHub] incubator-zeppelin pull request: Edit title fix for Notebook and P...

2015-12-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/512#issuecomment-163826549 Merging if there is 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 well.

[GitHub] incubator-zeppelin pull request: ZEPPELIN-268 : Adding Data Valida...

2015-12-10 Thread galleon
Github user galleon commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/262#issuecomment-163858912 @corneadoug have we measured the overhead in terms of memory/time? If that is a real issue and you believe this will damage the user experience then either

Re: [jira] [Created] (ZEPPELIN-311) Release 0.5.5

2015-12-10 Thread Corneau Damien
I can reply for the visualizations. I'm currently working on having ZEPPELIN-157 being merged. For the modular visualizations, it is in some way possible using %angular or %html. But we were at first thinking to have a set of core functions to facilitate bringing visualizations in the same way as

[GitHub] incubator-zeppelin pull request: ZEPPELIN-268 : Adding Data Valida...

2015-12-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/262#issuecomment-163866576 Haven't really be able to measure the overhead. But parsing the full query response, can be expensive if the data returned is big. One thing

[GitHub] incubator-zeppelin pull request: Add an Elasticsearch interpreter

2015-12-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/520#issuecomment-163836486 otherwise looks good, thanks! --- 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: Add/pr teamplate

2015-12-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/523 --- 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: R Interpreter for Zeppelin

2015-12-10 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-163856738 The current version of this is here: https://github.com/elbamos/Zeppelin-With-R --- If your project is set up for it, you can reply to this email and have