Repository: zeppelin
Updated Branches:
  refs/heads/branch-0.8 79aeabfeb -> 6d7b14e95


[ZEPPELIN-3228] Currently interpreter dependencies are not downloaded 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.carde...@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 3418055cca28aa16459d254999f580fd2785f5cc)
Signed-off-by: Jeff Zhang <zjf...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/6d7b14e9
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/6d7b14e9
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/6d7b14e9

Branch: refs/heads/branch-0.8
Commit: 6d7b14e95a1ea11a7752f0d2c3830c48c1553747
Parents: 79aeabf
Author: Cardenas, Jhon <ext-jhon.carde...@here.com>
Authored: Mon Feb 12 17:47:10 2018 -0500
Committer: Jeff Zhang <zjf...@apache.org>
Committed: Wed Feb 14 12:19:32 2018 +0800

----------------------------------------------------------------------
 .../apache/zeppelin/interpreter/InterpreterSettingManager.java | 6 ++++++
 1 file changed, 6 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zeppelin/blob/6d7b14e9/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSettingManager.java
----------------------------------------------------------------------
diff --git 
a/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSettingManager.java
 
b/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSettingManager.java
index bda1be6..04d4092 100644
--- 
a/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSettingManager.java
+++ 
b/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSettingManager.java
@@ -266,6 +266,12 @@ public class InterpreterSettingManager {
           this.interpreterRepositories.add(repo);
         }
       }
+
+      // force interpreter dependencies loading once the
+      // repositories have been loaded.
+      for (InterpreterSetting setting : interpreterSettings.values()) {
+        setting.setDependencies(setting.getDependencies());
+      }
     }
   }
 

Reply via email to