Skip to content

Conversation

@Leemoonsoo
Copy link
Member

What is this PR for?

While merging #697, NotebookRepo API changed.
And it breaks master branch. This PR implement checkpoint() method to fix it.

What type of PR is it?

Hot Fix

Todos

  • - Implement checkpoint()

Is there a relevant Jira issue?

ZEPPELIN-656

How should this be tested?

build master branch

Screenshots (if appropriate)

N/A

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

@khalidhuseynov
Copy link
Member

Thanks for catching up with latest API, looks good!

@Override
public void checkpoint(String noteId, String checkPointName) throws IOException {
// no-op
LOG.info("Checkpoint feature isn't suported in {}", this.getClass().toString());
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suported -> supported

@felixcheung
Copy link
Member

looks good

@granturing
Copy link
Contributor

looks good, thanks for handling that

@Leemoonsoo
Copy link
Member Author

Thanks guys for review.
I'm merging it as a hotfix

@asfgit asfgit closed this in 39417c0 Feb 19, 2016
prabhjyotsingh pushed a commit to prabhjyotsingh/zeppelin that referenced this pull request Feb 19, 2016
### What is this PR for?
While merging apache#697, NotebookRepo API changed.
And it breaks master branch. This PR implement checkpoint() method to fix it.

### What type of PR is it?
Hot Fix

### Todos
* [x] - Implement checkpoint()

### Is there a relevant Jira issue?
ZEPPELIN-656

### How should this be tested?
build master branch

### Screenshots (if appropriate)
N/A

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: Lee moon soo <moon@apache.org>

Closes apache#726 from Leemoonsoo/ZEPPELIN-656_hotfix and squashes the following commits:

6dbf024 [Lee moon soo] suported -> supported
3a61696 [Lee moon soo] Add checkpoint() method implementation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants