Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/53#issuecomment-98904182
Sorry for late responding and thanks again for contribution.
In my understanding, this branch trying to provides 1) security of
rest/websocket api. 2) multi-tenant environment for notebook save/load.
Because of this PR brings wide range of changes, we have couple of things
to discuss.
- providing login window
- displaying current user
- how to merge changes https://issues.apache.org/jira/browse/ZEPPELIN-26
- migration script for notebook directory structure change.
- notebook multi tenant model (per user / shares all, due to any user
actually can read any notebook from filesystem api on Spark/Pyspark interpreter)
I'm feeling like if it is possible to separate 1) security of
rest/websocket 2) multi-tenancy of notebook into different issue, it'll be
much faster to review and test.
What do you think?
---
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.
---
