Github user felixcheung commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/328#issuecomment-145189910
  
    I looked into this more.
    The default interpreter for a given group is the first interpreter:
    
https://github.com/apache/incubator-zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/NoteInterpreterLoader.java#L131
    ```
          // next, assume replName is 'group' of interpreter ('name' is 
ommitted)
          // search interpreter group and return first interpreter.
          for (InterpreterSetting setting : settings) {
            intpGroup = setting.getInterpreterGroup();
            Interpreter interpreter = intpGroup.get(0);
    ```
    In such case the `name` can be omitted and only the `group` is required.
    I think this matches the behavior here.
    
    It might be better if there is a way from a group of interpreters to decide 
which is the default interpreter rather then the first one in 
ZEPPELIN_INTERPRETERS 
(https://github.com/apache/incubator-zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java#L396),
 but that might over complicate the design.



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

Reply via email to