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

[astroshim]

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)

Questions:

• Does the licenses files need update? no
• Is there breaking changes for older versions? no
• Does this needs documentation? no

[astroshim]

re-trigger CI

[AhyoungRyu]

@astroshim Definitely in the latest master branch, I faced the below exception when I tried to run org.apache.zeppelin.interpreter.dev.ZeppelinApplicationDevServer in my IDE.

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 0
    at org.apache.zeppelin.interpreter.remote.RemoteInterpreterServer.main(RemoteInterpreterServer.java:136)

Maybe that's why you created this patch.

So I tried to test this PR with below steps

1. Build again with this patch: mvn clean package -DskipTests
2. Restart Zeppelin: ./bin/zeppelin-daemon.sh restart
3. Run org.apache.zeppelin.interpreter.dev.ZeppelinApplicationDevServer in my IntelliJ -> Successfully started
4. Run %spark new java.util.Date -> Nothing happened. I can't see the above Helium icons. Only Spark repl result was printed like res0: java.util.Date = Sun Aug 28 22:31:32 KST 2016. (I'm using Spark 2.0.)

Did I miss something? ㅠ_ㅠ

[astroshim]

@AhyoungRyu Thank you for reviewing!!
Did you clear your browser cache?

[AhyoungRyu]

@astroshim Yeah I did but still same :(

[AhyoungRyu]

@astroshim I rebuild with mvn clean package -DskipTests -Pexamples and it works. I should've attached -Pexamples. Now I can see the Helium icon :)

Anyway I checked ArrayIndexOutOfBoundsException is gone with this patch. LGTM 👍

[bzz]

Looks great to me!

Merging to master if there is no further discussion.

[astroshim]

ping

[corneadoug]

@astroshim You ping, I merge :)

[astroshim]

Thank you @corneadoug

[mikhaildubkov]

@astroshim ,

Once I checkout branch-0,6 I'm unable to build project due to compilation error:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project zeppelin-interpreter: Compilation failure
[ERROR] /zeppelin/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java:[35,43] package org.apache.zeppelin.interpreter.dev does not exist

Am I wrong with something or dev package really does not exists on branch-0.6 ?

[astroshim]

@mikhaildubkov dev doesn't exists on 0.6.x so you should use master branch for this.
Thanks.

[mikhaildubkov]

@astroshim i guess it would be better just replace ZeppelinDevServer.DEFAULT_TEST_INTERPRETER_PORT by its int value for branch-0.6 to avoid necessary change for project build.

[Leemoonsoo]

@corneadoug This PR shouldn't be merged to branch-0.6. Could you revert related commit from branch-0.6 ?

[corneadoug]

@Leemoonsoo Sure, I merged in 0.6 since it was the fixed version in JIRA

[astroshim]

Sorry maybe it's my mistake to set jira version.

[corneadoug]

@Leemoonsoo Done