You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by mi...@apache.org on 2017/01/26 05:05:13 UTC
zeppelin git commit: [HOTFIX][ZEPPELIN-2011] Add lib/interpreter to
classpath to fix interpreter installer
Repository: zeppelin
Updated Branches:
refs/heads/master 81dbf25d2 -> 90735979a
[HOTFIX][ZEPPELIN-2011] Add lib/interpreter to classpath to fix interpreter installer
### What is this PR for?
Interpreter installation fail with `NoClassDefFoundError` error when zeppelin is built as distribution package. This PR fixes it by adding `lib/interpreter` to classpath.
### What type of PR is it?
Bug Fix | Hot Fix
### What is the Jira issue?
[ZEPPELIN-2011](https://issues.apache.org/jira/browse/ZEPPELIN-2011)
### How should this be tested?
1. Build distribution package
```
mvn clean package -DskipTests -pl zeppelin-interpreter,zeppelin-zengine,:zeppelin-display_2.10,:zeppelin-spark-dependencies_2.10,:zeppelin-spark_2.10,zeppelin-web,zeppelin-server,zeppelin-distribution -am -Pbuild-distr
```
2. Change working directory
```
$ cd zeppelin-distribution/target/zeppelin-0.8.0-SNAPSHOT/zeppelin-0.8.0-SNAPSHOT
```
3. Run `bin/install-interpreter.sh` and see if error is gone.
### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no
Author: Mina Lee <mi...@apache.org>
Closes #1943 from minahlee/ZEPPELIN-2011 and squashes the following commits:
ed3776d [Mina Lee] Add lib/interpreter to classpath
Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/90735979
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/90735979
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/90735979
Branch: refs/heads/master
Commit: 90735979aaf4b2f58fa34ed0ab1aeccc37cb56bd
Parents: 81dbf25
Author: Mina Lee <mi...@apache.org>
Authored: Wed Jan 25 14:58:25 2017 +0900
Committer: Mina Lee <mi...@apache.org>
Committed: Thu Jan 26 14:05:06 2017 +0900
----------------------------------------------------------------------
bin/install-interpreter.sh | 1 +
1 file changed, 1 insertion(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/zeppelin/blob/90735979/bin/install-interpreter.sh
----------------------------------------------------------------------
diff --git a/bin/install-interpreter.sh b/bin/install-interpreter.sh
index 06be75c..d0c0394 100755
--- a/bin/install-interpreter.sh
+++ b/bin/install-interpreter.sh
@@ -40,6 +40,7 @@ fi
addJarInDir "${ZEPPELIN_HOME}/zeppelin-interpreter/target/lib"
addJarInDir "${ZEPPELIN_HOME}/lib"
+addJarInDir "${ZEPPELIN_HOME}/lib/interpreter"
CLASSPATH+=":${ZEPPELIN_CLASSPATH}"
$ZEPPELIN_RUNNER $JAVA_OPTS -cp $CLASSPATH $ZEPPELIN_INSTALL_INTERPRETER_MAIN ${@}