Skip to content

Conversation

@zjffdu
Copy link
Contributor

@zjffdu zjffdu commented Dec 21, 2020

What is this PR for?

The root cause is that we didn't reload the notes from NotebookRepo. It is a straightforwrad fix with only 1 line code change.

What type of PR is it?

[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]

Todos

  • - Task

What is the Jira issue?

How should this be tested?

  • Manually tested.

Screenshots (if appropriate)

Questions:

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

@Reamer
Copy link
Contributor

Reamer commented Dec 21, 2020

@zjffdu
The second commit in this PR should be separated into another PR.

@zjffdu
Copy link
Contributor Author

zjffdu commented Dec 21, 2020

Oops, my fault, let me fix it.

…e version when there's empty line ahead"

This reverts commit 3644538.
@zjffdu
Copy link
Contributor Author

zjffdu commented Jan 1, 2021

Will merge will if no more comment

@asfgit asfgit closed this in d0928ea Jan 3, 2021
asfgit pushed a commit that referenced this pull request Jan 3, 2021
### What is this PR for?

The root cause is that we didn't reload the notes from NotebookRepo. It is a straightforwrad fix with only 1 line code change.

### What type of PR is it?
[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-5162

### How should this be tested?
* Manually tested.

### Screenshots (if appropriate)

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

Author: Jeff Zhang <zjffdu@apache.org>

Closes #3997 from zjffdu/ZEPPELIN-5162 and squashes the following commits:

4aa5131 [Jeff Zhang] Revert "[ZEPPELIN-5169]. Hive set statement doesn't work for some hive version when there's empty line ahead"
3644538 [Jeff Zhang] [ZEPPELIN-5169]. Hive set statement doesn't work for some hive version when there's empty line ahead
f812548 [Jeff Zhang] [ZEPPELIN-5162]. Zeppelin reload note from storage doesn't work

(cherry picked from commit d0928ea)
Signed-off-by: Jeff Zhang <zjffdu@apache.org>
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