Skip to content

[ZEPPELIN-5303] Having colon in notebook name fails zeppelin to start#4106

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

[ZEPPELIN-5303] Having colon in notebook name fails zeppelin to start#4106
zjffdu wants to merge 1 commit intoapache:masterfrom
zjffdu:ZEPPELIN-5303

Conversation

@zjffdu
Copy link
Contributor

@zjffdu zjffdu commented May 3, 2021

What is this PR for?

The root cause in commons.vfs2, upgrading commons.vfs2 can fix this issue.

What type of PR is it?

[Bug Fix]

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

@zjffdu zjffdu force-pushed the ZEPPELIN-5303 branch 3 times, most recently from 73db3a1 to daf511e Compare May 3, 2021 14:20
Copy link
Contributor

@cuspymd cuspymd left a comment

Choose a reason for hiding this comment

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

Looks good!!

@zjffdu zjffdu force-pushed the ZEPPELIN-5303 branch 3 times, most recently from a85bac3 to 7c385b9 Compare May 5, 2021 00:37
@asfgit asfgit closed this in 60c5074 May 13, 2021
asfgit pushed a commit that referenced this pull request May 13, 2021
### What is this PR for?

The root cause in `commons.vfs2`, upgrading `commons.vfs2` can fix this issue.

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

### Todos
* [ ] - Task

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

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

b85f71b [Jeff Zhang] [ZEPPELIN-5303] Having colon in notebook name fails zeppelin to start

(cherry picked from commit 60c5074)
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