Skip to content

[ZEPPELIN-5112]. upgrade-note.sh throws exception while upgrading notes on hdfs storage#3958

Closed
zjffdu wants to merge 1 commit intoapache:masterfrom
zjffdu:ZEPPELIN-5112
Closed

[ZEPPELIN-5112]. upgrade-note.sh throws exception while upgrading notes on hdfs storage#3958
zjffdu wants to merge 1 commit intoapache:masterfrom
zjffdu:ZEPPELIN-5112

Conversation

@zjffdu
Copy link
Copy Markdown
Contributor

@zjffdu zjffdu commented Oct 29, 2020

What is this PR for?

The root cause is that we didn't put hadoop jars in the classpath in upgrade-note.sh. This PR update script upgrade-note.sh. Besides that this PR also fix an upgrading issue in ParagraphRuntimeInfo.java. The values is List in old version of zeppelin, but it is changed to List<Map<String, String>> in new version of zeppelin.

What type of PR is it?

[Bug Fix]

Todos

  • - Task

What is the Jira issue?

How should this be 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

@zjffdu
Copy link
Copy Markdown
Contributor Author

zjffdu commented Nov 3, 2020

Will merge if no more comment

@asfgit asfgit closed this in c1c0e40 Nov 4, 2020
asfgit pushed a commit that referenced this pull request Nov 4, 2020
…es on hdfs storage

### What is this PR for?

The root cause is that we didn't put hadoop jars in the classpath in `upgrade-note.sh`. This PR update script `upgrade-note.sh`. Besides that this PR also fix an upgrading issue in `ParagraphRuntimeInfo.java`. The `values` is List<String> in old version of zeppelin, but it is changed to List<Map<String, String>> in new version of zeppelin.

### What type of PR is it?
[Bug Fix]

### Todos
* [ ] - Task

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

### How should this be tested?
* CI pass
https://travis-ci.org/github/zjffdu/zeppelin/builds/739821895

### 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 #3958 from zjffdu/ZEPPELIN-5112 and squashes the following commits:

5e74dae [Jeff Zhang] [ZEPPELIN-5112]. upgrade-note.sh throws exception while upgrading notes on hdfs storage

(cherry picked from commit c1c0e40)
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.

1 participant