Skip to content

[ZEPPELIN-5728] Absence of cancelNote method in Zeppelin Client Api#4362

Merged
zjffdu merged 2 commits intoapache:masterfrom
huage1994:ZEPPELIN-5728
Apr 27, 2022
Merged

[ZEPPELIN-5728] Absence of cancelNote method in Zeppelin Client Api#4362
zjffdu merged 2 commits intoapache:masterfrom
huage1994:ZEPPELIN-5728

Conversation

@huage1994
Copy link
Copy Markdown
Contributor

What is this PR for?

Currently there's a cancelParagraph method in zeppelin client api, however, there's no cancelNote method.
PR is to add a cancelNote method in zeppelin client api.

What type of PR is it?

Improvement

Todos

  • - Task

What is the Jira issue?

How should this be tested?

  • CI passed

Screenshots (if appropriate)

Questions:

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

@huage1994
Copy link
Copy Markdown
Contributor Author

huage1994 commented Apr 21, 2022

😅 I just add a method in Zeppelin Client and do nothing else.
It's so strange that the CI tasks failed. And I re-run CI tasks twice in my own forked zeppelin repository, they still failed.

The log of failed CI tasks is about Jupyter kernel.

@zjffdu
Copy link
Copy Markdown
Contributor

zjffdu commented Apr 22, 2022

😅 I just add a method in Zeppelin Client and do nothing else. It's so strange that the CI tasks failed. And I re-run CI tasks twice in my own forked zeppelin repository, they still failed.

The log of failed CI tasks is about Jupyter kernel.

This error is not related with your PR. It also happens in master

@zjffdu
Copy link
Copy Markdown
Contributor

zjffdu commented Apr 23, 2022

@huage1994 CI issue is fixed, please rebase your PR

@huage1994
Copy link
Copy Markdown
Contributor Author

Thanks @zjffdu a lot. I have added test and rebased my PR.

@zjffdu
Copy link
Copy Markdown
Contributor

zjffdu commented Apr 26, 2022

CI is passed, will merge if no more comment

@zjffdu zjffdu merged commit 8672d6c into apache:master Apr 27, 2022
prabhjyotsingh pushed a commit to prabhjyotsingh/zeppelin that referenced this pull request May 2, 2022
…pache#4362)

* [ZEPPELIN-5728] Absence of cancelNote method in Zeppelin Client Api

* add test
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.

2 participants