GitHub user khalidhuseynov reopened a pull request:

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

    Update and refactor NotebookRepo versioning API

    ### What is this PR for?
    This is firstly to refactor API for versioning and keep everthing inside of 
one interface (NotebookRepo) instead of two different interfaces 
(NotebookRepoVersioned). Secondly, there're modifications to existing 
versioning api, with considerations of future complete implementation of 
versioning. Note that this PR doesn't implement all suggested interfaces, but 
lays foundation for their implementation.
    
    ### What type of PR is it?
    Improvement && Refactoring
    
    ### Todos
    * [x] - move versioning api (get, history) from NotebookRepoVersioned to 
NotebookRepo
    * [x] - refactor and naming changes
    * [x] - modify checkpoint api (add return value) and modify 
NotebookRepoSync to deal with it
    
    
    ### What is the Jira issue?
    
    
    ### How should this be tested?
    Basically it doesn't add new functionality, so the only requirement is for 
tests to pass.
    
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the licenses files need update? No
    * Is there breaking changes for older versions? not breaking, but some api 
changes
    * Does this needs documentation? No


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

    $ git pull https://github.com/khalidhuseynov/incubator-zeppelin 
repo/versioning-api-update

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

    https://github.com/apache/zeppelin/pull/1007.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 #1007
    
----
commit faf1b5b06a0753fa728caec7579f1bb6118137ae
Author: Khalid Huseynov <khalid...@nflabs.com>
Date:   2016-06-13T22:10:03Z

    move and update versioning api
    
    first of all versioning api was moved into NotebookRepo to keep all api in 
one place, secondly it was updated with Rev class referring to revision unit

commit 3dce38bcd4b7a1eeb110e336d069527028a7429c
Author: Khalid Huseynov <khalid...@nflabs.com>
Date:   2016-06-13T22:14:12Z

    propagate changes to all repos

commit 1ec67e648afb39cbc50ba1b98824c8a86e70828a
Author: Khalid Huseynov <khalid...@nflabs.com>
Date:   2016-06-13T22:14:46Z

    apply changes to NotebookRepoSync

commit ee1eeb3357af4a051625d7058819c7e3aad3be8d
Author: Khalid Huseynov <khalid...@nflabs.com>
Date:   2016-06-13T22:15:24Z

    fix tests

commit 6472888c7982d21710eb7dcea2aa9527d271ad35
Author: Khalid Huseynov <khalid...@nflabs.com>
Date:   2016-06-13T23:18:55Z

    fix checkstyle

commit 141f5df8dfd71199d9000281f1e9d1da1f7c3752
Author: Khalid Huseynov <khalid...@nflabs.com>
Date:   2016-06-14T01:49:30Z

    Merge branch 'master' into repo/versioning-api-update

commit f084b3fe1352f6569b936e6553ad4e2512731c62
Author: Khalid Huseynov <khalid...@nflabs.com>
Date:   2016-06-14T18:18:22Z

    Rev -> Revision

----


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