[ZEPPELIN-3228] Currently interpreter dependencies are not downloaded on zeppelin start - regression issue #2792
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
LGTM |
|
@jhonderson ZEPPELIN-1143 has been fixed. Is this a regression issue ? If so, could you create another ticket for this issue ? |
jhonderson
changed the title
|
@zjffdu i have created the ticket for this issue. If this is approved, could you please merge this solution to branch-0.8? |
|
Thanks @jhonderson , will merge it to branch-0.8 if CI pass |
|
@jhonderson Can you retrigger the build (via close and reopen), I can not access your jenkins job |
|
Re-triggering build. |
|
@zjffdu by the way, here is my travis: https://travis-ci.org/jhonderson/zeppelin/builds/340723065 |
asfgit
pushed a commit
that referenced
this pull request
Feb 14, 2018
… on zeppelin start - regression issue Currently interpreter dependencies are not downloaded on zeppelin start. This was solved in [ZEPPELIN-3228], but it is happening again. ### What is this PR for? When zeppelin is started/restarted, server should try and download interpreter dependencies. ### What type of PR is it? [Bug Fix] ### What is the Jira issue? * [ZEPPELIN-3228] ### How should this be tested? * Put a dependency (say "org.apache.commons:commons-csv:1.1") in any of the interpreter. * From command line delete local-repo directory * Restart zeppelin server Expectation is local-repo should be recreated with all the dependencies that were mentioned in any of the interpreters. ### Questions: * Does the licenses files need update? n/a * Is there breaking changes for older versions? n/a * Does this needs documentation? n/a Author: Cardenas, Jhon <ext-jhon.cardenas@here.com> Closes #2792 from jhonderson/ZEPPELIN-1143 and squashes the following commits: 3fcefb2 [Cardenas, Jhon] [ZEPPELIN-1143] When zeppelin starts it does the interpreter dependencies loading. (cherry picked from commit 3418055) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
necosta
pushed a commit
to nokia/zeppelin
that referenced
this pull request
Feb 14, 2018
… on zeppelin start - regression issue Currently interpreter dependencies are not downloaded on zeppelin start. This was solved in [ZEPPELIN-3228], but it is happening again. When zeppelin is started/restarted, server should try and download interpreter dependencies. [Bug Fix] * [ZEPPELIN-3228] * Put a dependency (say "org.apache.commons:commons-csv:1.1") in any of the interpreter. * From command line delete local-repo directory * Restart zeppelin server Expectation is local-repo should be recreated with all the dependencies that were mentioned in any of the interpreters. * Does the licenses files need update? n/a * Is there breaking changes for older versions? n/a * Does this needs documentation? n/a Author: Cardenas, Jhon <ext-jhon.cardenas@here.com> Closes apache#2792 from jhonderson/ZEPPELIN-1143 and squashes the following commits: 3fcefb2 [Cardenas, Jhon] [ZEPPELIN-1143] When zeppelin starts it does the interpreter dependencies loading.
jwagun
pushed a commit
to jwagun/zeppelin
that referenced
this pull request
Apr 23, 2018
… on zeppelin start - regression issue Currently interpreter dependencies are not downloaded on zeppelin start. This was solved in [ZEPPELIN-3228], but it is happening again. ### What is this PR for? When zeppelin is started/restarted, server should try and download interpreter dependencies. ### What type of PR is it? [Bug Fix] ### What is the Jira issue? * [ZEPPELIN-3228] ### How should this be tested? * Put a dependency (say "org.apache.commons:commons-csv:1.1") in any of the interpreter. * From command line delete local-repo directory * Restart zeppelin server Expectation is local-repo should be recreated with all the dependencies that were mentioned in any of the interpreters. ### Questions: * Does the licenses files need update? n/a * Is there breaking changes for older versions? n/a * Does this needs documentation? n/a Author: Cardenas, Jhon <ext-jhon.cardenas@here.com> Closes apache#2792 from jhonderson/ZEPPELIN-1143 and squashes the following commits: 3fcefb2 [Cardenas, Jhon] [ZEPPELIN-1143] When zeppelin starts it does the interpreter dependencies loading.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently interpreter dependencies are not downloaded on zeppelin start. This was solved in [ZEPPELIN-3228], but it is happening again.
What is this PR for?
When zeppelin is started/restarted, server should try and download interpreter dependencies.
What type of PR is it?
[Bug Fix]
What is the Jira issue?
How should this be tested?
Expectation is local-repo should be recreated with all the dependencies that were mentioned in any of the interpreters.
Questions: