Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/47#issuecomment-97295608
Hi, @cfmcgrady. Thanks for contribution.
Constructor type for NotebookRepo is slightly changed.
from NotebookRepo(ZeppelinConfiguration conf, URI uri) to
NotebookRepo(ZeppelinConfiguration conf). Please rebase this branch with master
branch.
One concern is
Currently zeppelin-daemon process and interpreter process uses common.sh to
construct it's classpath.
Once zeppelin-daemon have HDFS dependencies, then by common.sh, it'll be
also loaded into every interpreter process. Especially spark include HDFS
related dependency, too, they'll conflict.
And version of HDFS related dependency's is fixed. i think it'll be better
if it is 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 [email protected] or file a JIRA ticket
with INFRA.
---
