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 <prabhjyotsi...@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 <prabhjyotsi...@gmail.com>
Authored: Tue Mar 21 14:49:30 2017 +0530
Committer: Prabhjyot Singh <prabhjyotsi...@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();

Reply via email to