You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by tejasapatil <gi...@git.apache.org> on 2015/10/27 04:25:52 UTC

[GitHub] incubator-zeppelin pull request: ZEPPELIN-364 Make interpreter set...

GitHub user tejasapatil opened a pull request:

    https://github.com/apache/incubator-zeppelin/pull/371

    ZEPPELIN-364 Make interpreter settings file path configurable

    

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

    $ git pull https://github.com/tejasapatil/incubator-zeppelin ZEPPELIN-364

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

    https://github.com/apache/incubator-zeppelin/pull/371.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 #371
    
----
commit 0a2b56d9cb7cffe51d8e383604e4f4c63a6065a7
Author: Tejas Patil <te...@fb.com>
Date:   2015-10-27T03:18:52Z

    ZEPPELIN-364 Make interpreter settings file path configurable

----


---
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] incubator-zeppelin pull request: ZEPPELIN-364 Make interpreter set...

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

    https://github.com/apache/incubator-zeppelin/pull/371


---
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] incubator-zeppelin pull request: ZEPPELIN-364 Make interpreter set...

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/371#issuecomment-153982467
  
    One difference of interpreter.json is it's being updated from Zeppelin. other configuration files under conf/ are read only. So this PR might be useful for some cases.


---
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] incubator-zeppelin pull request: ZEPPELIN-364 Make interpreter set...

Posted by felixcheung <gi...@git.apache.org>.
Github user felixcheung commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/371#issuecomment-152009968
  
    There are other configuration though: 
    log4j.properties, zeppelin-site.xml, zeppelin-env.sh, interpreter.json
    
    Why not just push the config with Chef/Puppet to /conf if this is to help with config management?


---
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.
---