Github user felixcheung commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/578#issuecomment-170776132
  
    @corneadoug  looks good, I'd only add in 2, instead of a specific line 
number, could we refer to it as:
    Add your interpreter to the `ZEPPELIN_INTERPRETERS()` list in 
[zeppelin-zengine/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java](https://github.com/apache/incubator-zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java)


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to