You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by co...@apache.org on 2016/09/01 07:27:47 UTC

zeppelin git commit: [ZEPPELIN-1365] Error of Zeppelin Application in development mode.

Repository: zeppelin
Updated Branches:
  refs/heads/master d93fb7361 -> c580a82ad


[ZEPPELIN-1365] Error of Zeppelin Application in development mode.

### What is this PR for?
This PR fixes the bug of running application in development mode.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1365

### How should this be tested?
1. run zeppelin
2. run org.apache.zeppelin.interpreter.dev.ZeppelinApplicationDevServer in development mode.
(http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/development/writingzeppelinapplication.html)
3. and run paragraph like screenshot.

### Screenshots (if appropriate)
![image](https://cloud.githubusercontent.com/assets/3348133/17935588/d95ec2de-6a56-11e6-84d9-19030984411d.png)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: astroshim <hs...@nflabs.com>

Closes #1358 from astroshim/ZEPPELIN-1365 and squashes the following commits:

9f640e5 [astroshim] add argument check.


Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/c580a82a
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/c580a82a
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/c580a82a

Branch: refs/heads/master
Commit: c580a82ad54a5f165dffa2c590c1991375fffb75
Parents: d93fb73
Author: astroshim <hs...@nflabs.com>
Authored: Wed Aug 24 23:52:45 2016 +0900
Committer: Damien CORNEAU <co...@gmail.com>
Committed: Thu Sep 1 16:27:41 2016 +0900

----------------------------------------------------------------------
 .../zeppelin/interpreter/remote/RemoteInterpreterServer.java  | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zeppelin/blob/c580a82a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java
----------------------------------------------------------------------
diff --git a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java
index 1730265..57dd768 100644
--- a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java
+++ b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java
@@ -34,6 +34,7 @@ import org.apache.zeppelin.display.*;
 import org.apache.zeppelin.helium.*;
 import org.apache.zeppelin.interpreter.*;
 import org.apache.zeppelin.interpreter.InterpreterResult.Code;
+import org.apache.zeppelin.interpreter.dev.ZeppelinDevServer;
 import org.apache.zeppelin.interpreter.thrift.*;
 import org.apache.zeppelin.resource.*;
 import org.apache.zeppelin.scheduler.Job;
@@ -133,7 +134,11 @@ public class RemoteInterpreterServer
 
   public static void main(String[] args)
       throws TTransportException, InterruptedException {
-    int port = Integer.parseInt(args[0]);
+
+    int port = ZeppelinDevServer.DEFAULT_TEST_INTERPRETER_PORT;
+    if (args.length > 0) {
+      port = Integer.parseInt(args[0]);
+    }
     RemoteInterpreterServer remoteInterpreterServer = new RemoteInterpreterServer(port);
     remoteInterpreterServer.start();
     remoteInterpreterServer.join();