Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 76C57200BB3 for ; Wed, 19 Oct 2016 02:44:16 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 75770160AF7; Wed, 19 Oct 2016 00:44:16 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id B73B1160AE5 for ; Wed, 19 Oct 2016 02:44:15 +0200 (CEST) Received: (qmail 9042 invoked by uid 500); 19 Oct 2016 00:44:14 -0000 Mailing-List: contact commits-help@zeppelin.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@zeppelin.apache.org Delivered-To: mailing list commits@zeppelin.apache.org Received: (qmail 9033 invoked by uid 99); 19 Oct 2016 00:44:14 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 19 Oct 2016 00:44:14 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id B86AAE00DC; Wed, 19 Oct 2016 00:44:14 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: minalee@apache.org To: commits@zeppelin.apache.org Message-Id: <5c7bf9f697b94ad49db264ff1599f85b@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: zeppelin git commit: [ZEPPELIN-1534] Does not load dependency library when creating new interpreter. Date: Wed, 19 Oct 2016 00:44:14 +0000 (UTC) archived-at: Wed, 19 Oct 2016 00:44:16 -0000 Repository: zeppelin Updated Branches: refs/heads/master 1214463c6 -> abf0470a3 [ZEPPELIN-1534] Does not load dependency library when creating new interpreter. ### What is this PR for? This PR fixes dependency library loading bug when creating new interpreter. ### What type of PR is it? Bug Fix ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-1534 ### How should this be tested? 1. Create new JDBC interpreter refer to http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/interpreter/jdbc.html#mysql. 2. Create new paragraph for testing new JDBC interpreter. 3. Run paragraph. ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no Author: astroshim Closes #1507 from astroshim/ZEPPELIN-1534 and squashes the following commits: aebe352 [astroshim] add loadInterpreterDependencies function in createNewSetting() Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/abf0470a Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/abf0470a Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/abf0470a Branch: refs/heads/master Commit: abf0470a3c8f3f7d38bc96c345f72f3298243f5c Parents: 1214463 Author: astroshim Authored: Tue Oct 11 23:04:27 2016 +0900 Committer: Mina Lee Committed: Wed Oct 19 09:44:08 2016 +0900 ---------------------------------------------------------------------- .../java/org/apache/zeppelin/interpreter/InterpreterFactory.java | 1 + 1 file changed, 1 insertion(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/zeppelin/blob/abf0470a/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterFactory.java ---------------------------------------------------------------------- diff --git a/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterFactory.java b/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterFactory.java index 494361b..5d9f301 100644 --- a/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterFactory.java +++ b/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterFactory.java @@ -596,6 +596,7 @@ public class InterpreterFactory implements InterpreterGroupFactory { setting.setProperties(p); setting.setInterpreterGroupFactory(this); interpreterSettings.put(setting.getId(), setting); + loadInterpreterDependencies(setting); saveToFile(); return setting; }