You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by pr...@apache.org on 2017/03/22 16:00:42 UTC
zeppelin git commit: [HOTFIX][ZEPPELIN-2149] correct variable name
used in interpreter.sh
Repository: zeppelin
Updated Branches:
refs/heads/master 29dc3da1e -> 4a51028e9
[HOTFIX][ZEPPELIN-2149] correct variable name used in interpreter.sh
### What is this PR for?
This fixes the typo is https://github.com/apache/zeppelin/pull/2107.
### What type of PR is it?
[Bug Fix]
### How should this be tested?
Create multiple interpreters (say shell interpreter) with different names (say sh1, sh2, etc.).
Now on running these different interpreters should log to different files.
### Screenshots (if appropriate)
### 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: Prabhjyot Singh <pr...@gmail.com>
Closes #2170 from prabhjyotsingh/ZEPPELIN-2149_v2 and squashes the following commits:
7baf81a [Prabhjyot Singh] [ZEPPELIN-2149] correct variable name used in interpreter.sh
Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/4a51028e
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/4a51028e
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/4a51028e
Branch: refs/heads/master
Commit: 4a51028e9ef9dfaee76a85eb2f6c862afa5deac7
Parents: 29dc3da
Author: Prabhjyot Singh <pr...@gmail.com>
Authored: Tue Mar 21 14:49:30 2017 +0530
Committer: Prabhjyot Singh <pr...@gmail.com>
Committed: Wed Mar 22 21:29:19 2017 +0530
----------------------------------------------------------------------
bin/interpreter.sh | 2 +-
.../interpreter/remote/RemoteInterpreterManagedProcess.java | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/zeppelin/blob/4a51028e/bin/interpreter.sh
----------------------------------------------------------------------
diff --git a/bin/interpreter.sh b/bin/interpreter.sh
index 5565ea9..a9d8f32 100755
--- a/bin/interpreter.sh
+++ b/bin/interpreter.sh
@@ -23,7 +23,7 @@ function usage() {
echo "usage) $0 -p <port> -d <interpreter dir to load> -l <local interpreter repo dir to load> -g <interpreter group name>"
}
-while getopts "hp:d:l:v:u:n:" o; do
+while getopts "hp:d:l:v:u:g:" o; do
case ${o} in
h)
usage
http://git-wip-us.apache.org/repos/asf/zeppelin/blob/4a51028e/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterManagedProcess.java
----------------------------------------------------------------------
diff --git a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterManagedProcess.java b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterManagedProcess.java
index 23bfb84..1fb9b90 100644
--- a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterManagedProcess.java
+++ b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterManagedProcess.java
@@ -112,7 +112,7 @@ public class RemoteInterpreterManagedProcess extends RemoteInterpreterProcess
}
cmdLine.addArgument("-l", false);
cmdLine.addArgument(localRepoDir, false);
- cmdLine.addArgument("-n", false);
+ cmdLine.addArgument("-g", false);
cmdLine.addArgument(interpreterGroupName, false);
executor = new DefaultExecutor();