GitHub user khalidhuseynov reopened a pull request:

    https://github.com/apache/zeppelin/pull/1625

    [ZEPPELIN-1657] Private/public mode for user note creation/import

    ### What is this PR for?
    In multi-user environment when users create a notebook normally they would 
expect that notebook be listed only in their list/workbench. Currently Zeppelin 
creates/imports notes as public by default. There should be at least a way to 
pass through configuration to make this behaviour private be default. Should 
discuss whether it's public or private by default.
    
    
    ### What type of PR is it?
    Improvement
    
    ### Todos
    * [x] - set permissions on create/import
    * [x] - test
    * [x] - review, feedback, decide whether public/private by default
    * [x] - pass through env. config
    
    ### What is the Jira issue?
    [ZEPPELIN-1657](https://issues.apache.org/jira/browse/ZEPPELIN-1657)
    
    ### How should this be tested?
    1. set `zeppelin.notebook.public` property as false in `zeppelin-site.xml`
    2. login as user1 and create noteA, note should appear in your list of notes
    3. logout and login as user2, shouldn't be able to see noteA
    
    ### Screenshots (if appropriate)
    TBD
    
    ### Questions:
    * Does the licenses files need update? no
    * Is there breaking changes for older versions? no
    * Does this needs documentation? maybe


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/khalidhuseynov/incubator-zeppelin 
feat/public-note-create

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zeppelin/pull/1625.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1625
    
----
commit b7f28b30616419d22b62d116f70568821c6f3711
Author: Khalid Huseynov <khalid...@gmail.com>
Date:   2016-11-11T09:13:18Z

    add property to site.xml

commit 4062af09c9a585f7a9dd164bf3d8b90ceb3bde75
Author: Khalid Huseynov <khalid...@gmail.com>
Date:   2016-11-11T10:03:54Z

    add var to env.sh

commit 97324096f674d61dee732561d986db728985d2fa
Author: Khalid Huseynov <khalid...@gmail.com>
Date:   2016-11-11T10:52:23Z

    add variable to ZeppelinConfiguration

commit 44297a40fd67ce3fe25756417091e4fa6ccf9176
Author: Khalid Huseynov <khalid...@gmail.com>
Date:   2016-11-13T13:15:13Z

    set new note permissions

commit 28abffac6a4f6fd8514fdea706af12a42958406e
Author: Khalid Huseynov <khalid...@gmail.com>
Date:   2016-11-13T13:15:51Z

    set permissions on note create and import

commit 496b80c0823494d9c532574bcd69538b8bb2a3d8
Author: Khalid Huseynov <khalid...@gmail.com>
Date:   2016-11-13T16:57:51Z

    read env var from conf file directly

commit 1a945e8e183ec1baab807b5c9ce522274093b061
Author: Khalid Huseynov <khalid...@gmail.com>
Date:   2016-11-13T16:58:04Z

    add test

commit 876b79880ad0426f691e390c5e8b25baef35b380
Author: Khalid Huseynov <khalid...@gmail.com>
Date:   2016-11-14T03:39:40Z

    fix vfs testSave npe

commit f862ae1e875d51c18c353b9eda9cd2d0734f5a12
Author: Khalid Huseynov <khalid...@gmail.com>
Date:   2016-11-14T04:30:01Z

    fix InterpreterFactory test

commit fc91b2f15725e91db1cbdf0439e8dd8779fc4f68
Author: Khalid Huseynov <khalid...@gmail.com>
Date:   2016-11-16T08:14:44Z

    add description to install config

commit f2a4fe6d884540f318947289f83f98390f2af227
Author: Khalid Huseynov <khalid...@gmail.com>
Date:   2016-11-17T05:53:10Z

    fix test: set state back

----


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