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();