Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ZEPPELIN-3321] isRevisionSupported not working for importing notebooks. #2862

Closed
wants to merge 3 commits into from

Conversation

prabhjyotsingh
Copy link
Contributor

@prabhjyotsingh prabhjyotsingh commented Mar 13, 2018

What is this PR for?

when importing note of from one zeppelin instance that don't support revision to another zeppelin instance where revision is supported, the imported note still don't support revision.

What type of PR is it?

[Bug Fix]

What is the Jira issue?

Questions:

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

@prabhjyotsingh
Copy link
Contributor Author

@zjffdu, @r-kamath can you help review this.

@r-kamath
Copy link
Member

@prabhjyotsingh please add a test

Change-Id: I2ba8a49b79ae94212046933ee74d8fafb3a387bb
@zjffdu
Copy link
Contributor

zjffdu commented Mar 14, 2018

LGTM

1 similar comment
@r-kamath
Copy link
Member

LGTM

@prabhjyotsingh
Copy link
Contributor Author

Thanks for the review will merge this if no more discussion.

asfgit pushed a commit that referenced this pull request Mar 15, 2018
 when importing note of from one zeppelin instance that don't support revision to another zeppelin instance where revision is supported, the imported note still don't support revision.

[Bug Fix]

* [ZEPPELIN-3321](https://issues.apache.org/jira/browse/ZEPPELIN-3321)

* Does the licenses files need update? N/A
* Is there breaking changes for older versions? N/A
* Does this needs documentation? N/A

Author: Prabhjyot Singh <prabhjyotsingh@gmail.com>

Closes #2862 from prabhjyotsingh/ZEPPELIN-3321 and squashes the following commits:

eca6b59 [Prabhjyot Singh] add unit test for isRevisionSupported
552fdeb [Prabhjyot Singh] approach 2: set as isRevisionSupported zeppelin global config
18873d3 [Prabhjyot Singh] ZEPPELIN-3321: fix isRevisionSupported

Change-Id: I49ec3350303b39a41b769bc47941c3397160234c
(cherry picked from commit 6001b98)
Signed-off-by: Prabhjyot Singh <prabhjyotsingh@gmail.com>
@asfgit asfgit closed this in 6001b98 Mar 15, 2018
jwagun pushed a commit to jwagun/zeppelin that referenced this pull request Apr 23, 2018
 when importing note of from one zeppelin instance that don't support revision to another zeppelin instance where revision is supported, the imported note still don't support revision.

[Bug Fix]

* [ZEPPELIN-3321](https://issues.apache.org/jira/browse/ZEPPELIN-3321)

* Does the licenses files need update? N/A
* Is there breaking changes for older versions? N/A
* Does this needs documentation? N/A

Author: Prabhjyot Singh <prabhjyotsingh@gmail.com>

Closes apache#2862 from prabhjyotsingh/ZEPPELIN-3321 and squashes the following commits:

eca6b59 [Prabhjyot Singh] add unit test for isRevisionSupported
552fdeb [Prabhjyot Singh] approach 2: set as isRevisionSupported zeppelin global config
18873d3 [Prabhjyot Singh] ZEPPELIN-3321: fix isRevisionSupported

Change-Id: I49ec3350303b39a41b769bc47941c3397160234c
prabhjyotsingh added a commit to prabhjyotsingh/zeppelin that referenced this pull request Jul 4, 2018
 when importing note of from one zeppelin instance that don't support revision to another zeppelin instance where revision is supported, the imported note still don't support revision.

[Bug Fix]

* [ZEPPELIN-3321](https://issues.apache.org/jira/browse/ZEPPELIN-3321)

* Does the licenses files need update? N/A
* Is there breaking changes for older versions? N/A
* Does this needs documentation? N/A

Author: Prabhjyot Singh <prabhjyotsingh@gmail.com>

Closes apache#2862 from prabhjyotsingh/ZEPPELIN-3321 and squashes the following commits:

eca6b59 [Prabhjyot Singh] add unit test for isRevisionSupported
552fdeb [Prabhjyot Singh] approach 2: set as isRevisionSupported zeppelin global config
18873d3 [Prabhjyot Singh] ZEPPELIN-3321: fix isRevisionSupported

Change-Id: I49ec3350303b39a41b769bc47941c3397160234c
(cherry picked from commit 6001b98)
Signed-off-by: Prabhjyot Singh <prabhjyotsingh@gmail.com>
(cherry picked from commit 257db1d)
@prabhjyotsingh prabhjyotsingh deleted the ZEPPELIN-3321 branch May 2, 2022 16:27
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.

3 participants