Github user bzz commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/123#issuecomment-117382324
  
    I think right now, with default configuration, Zeppelin is only going to 
use local storage, as before, but in case user put new value to  
`ConfVars.ZEPPELIN_NOTEBOOK_STORAGE` - it will only then sync between local and 
a new one.
    
    @guptarajat @Leemoonsoo will this be enough, or do you think we need an 
explicit `feature toggle` for the same behaviour?
    
    
    @khalidhuseynov adding tests for `NotebookRepoSync`, to cover the 
implemented cases is really a must.



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