[GitHub] zeppelin issue #1581: Ignite versions fixed

2016-11-09 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1581 @agura Thanks for taking care of the Ignite documentation. The update is looks good to me. Merge into master if there are no more discussions. --- If your project is set up for it, you

[GitHub] zeppelin issue #1581: Ignite versions fixed

2016-11-07 Thread agura
Github user agura commented on the issue: https://github.com/apache/zeppelin/pull/1581 @AhyoungRyu Are there something to do in order to merge 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 project does

[GitHub] zeppelin issue #1581: Ignite versions fixed

2016-11-07 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1581 @agura `../manual/interpreters.html#what-is-interpreter-setting` will be connected to [https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/manual/interpreters.html#what-is-interpreter-setting](http

[GitHub] zeppelin issue #1581: Ignite versions fixed

2016-11-07 Thread agura
Github user agura commented on the issue: https://github.com/apache/zeppelin/pull/1581 @AhyoungRyu I don't see the `../manual/interpreters.html#what-is-interpreter-setting` link. Where is it? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1581: Ignite versions fixed

2016-11-03 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1581 @agura Thanks for your quick response. Except for `../manual/interpreters.html#what-is-interpreter-setting` link, LGTM. --- If your project is set up for it, you can reply to this email and h