[GitHub] incubator-zeppelin pull request: Lens Interpreter

2016-02-24 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/100#issuecomment-188639456 @praagarw Could you please this PR? --- 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: Lens Interpreter

2015-07-20 Thread praagarw
Github user praagarw commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/100#issuecomment-122827553 Fixed the build failure is pull request https://github.com/apache/incubator-zeppelin/pull/159 --- If your project is set up for it, you can reply to this e

[GitHub] incubator-zeppelin pull request: Lens Interpreter

2015-07-19 Thread btiernay
Github user btiernay commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/100#issuecomment-122734763 Yep, I'm getting the same on master --- 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: Lens Interpreter

2015-07-18 Thread simhan
Github user simhan commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/100#issuecomment-122626347 getting following error with latest build . Zeppelin: Apache Ignite interpreter SUCCESS [ 0.370 s] [INFO] Zeppelin: Lens interpreter .

[GitHub] incubator-zeppelin pull request: Lens Interpreter

2015-06-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/100#issuecomment-114249575 @praagarw I have clean up the branch and created new PR #114. If you think everything is good, let's close this PR and merge #114 --- If your project is

[GitHub] incubator-zeppelin pull request: Lens Interpreter

2015-06-21 Thread praagarw
Github user praagarw commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/100#issuecomment-113995129 @Leemoonsoo I added org.apache.zeppelin.lens.LensInterpreter to conf/zeppelin-site.xml.template. Howerver tests are failing. Can you please take this forw

[GitHub] incubator-zeppelin pull request: Lens Interpreter

2015-06-20 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/100#issuecomment-113836069 @praagarw, Everything looks good and only missing thing i think is adding `org.apache.zeppelin.lens.LensInterpreter` into conf/zeppelin-site.xml.template.

[GitHub] incubator-zeppelin pull request: Lens Interpreter

2015-06-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/100#issuecomment-112093766 This is really great work! One comment, you probably want to add Lens interpreter in conf/zeppelin-site.xml.template, zeppelin.interpreters proper

[GitHub] incubator-zeppelin pull request: Lens Interpreter

2015-06-12 Thread praagarw
GitHub user praagarw opened a pull request: https://github.com/apache/incubator-zeppelin/pull/100 Lens Interpreter You can merge this pull request into a Git repository by running: $ git pull https://github.com/praagarw/incubator-zeppelin master Alternatively you can review a