[GitHub] zeppelin issue #1042: [WIP] bin/install-interpreter.sh for netinst package

2016-06-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1042
  
@prabhjyotsingh I think that's good idea.

If this script runs before start Zeppelin, right after download and 
unarchive, Zeppelin will auto create conf/interpreter.json with all installed 
interpreters. 

If this script runs after conf/interpreter.json created, it'll not update 
conf/interpreter.json. And i think it make sense while conf/interpreter.json 
might customized by user at this time, and user may not want to tough/add 
something into it.

I think we'll have better way to install interpreter once Zeppelin have 
dynamic interpreter loading capability 
(https://issues.apache.org/jira/browse/ZEPPELIN-598)

@felixcheung I have removed hardcoded list and updated to read list from 
text file `conf/interpreter-list`.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1042: [WIP] bin/install-interpreter.sh for netinst package

2016-06-20 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1042
  
Both suggestions are nice addition, and there is also a refactoring on 
TODOs so I guess it's WIP now.

👍  for making it part of the release, as discussed dev@z.a.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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1042: [WIP] bin/install-interpreter.sh for netinst package

2016-06-18 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1042
  
One more feature request for this, with downloading an interpreter, 
creation/insertion of relevant entry in `conf/interpreter.json`


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---