You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by agura <gi...@git.apache.org> on 2016/11/02 17:21:26 UTC

[GitHub] zeppelin pull request #1581: Ignite versions fixed

GitHub user agura opened a pull request:

    https://github.com/apache/zeppelin/pull/1581

    Ignite versions fixed

    ### What is this PR for?
    This PR fixes Apache Ignite version mentions in interpreter documentation.
    
    ### What type of PR is it?
    [Improvement]

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/agura/incubator-zeppelin ignite-docs

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zeppelin/pull/1581.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1581
    
----
commit d8710b3c200fb517469313495a11728c12da73c2
Author: agura <ag...@gridgain.com>
Date:   2016-11-02T17:12:27Z

    Ignite versions fixed

----


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1581: Ignite versions fixed

Posted by AhyoungRyu <gi...@git.apache.org>.
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 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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1581: Ignite versions fixed

Posted by agura <gi...@git.apache.org>.
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 not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1581: Ignite versions fixed

Posted by AhyoungRyu <gi...@git.apache.org>.
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](https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/manual/interpreters.html#what-is-interpreter-setting).


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #1581: Ignite versions fixed

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/zeppelin/pull/1581


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1581: Ignite versions fixed

Posted by agura <gi...@git.apache.org>.
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 on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1581: Ignite versions fixed

Posted by AhyoungRyu <gi...@git.apache.org>.
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 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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---