zeppelin git commit: [MINOR] Verify appId is not null in LivyInterpreterIT

[zj...@apache.org]

Repository: zeppelin
Updated Branches:
  refs/heads/master 48647e597 -> 6fc3057d5


[MINOR] Verify appId is not null in LivyInterpreterIT

### What is this PR for?
Someone complain that they could not get appId, this PR just try to verify appId returned by livy rest api is not null.

### What type of PR is it?
[Improvement]

### Todos
* [ ] - Task

### What is the Jira issue?
* No jira created

### How should this be tested?
* CI pass

### Screenshots (if appropriate)

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

Author: Jeff Zhang <zj...@apache.org>

Closes #2999 from zjffdu/minor_livy and squashes the following commits:

eae0cf58c [Jeff Zhang] [MINOR] Verify appId is null in LivyInterpreterIT


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

Branch: refs/heads/master
Commit: 6fc3057d529691e697361ef9a67e21cac75e21ed
Parents: 48647e5
Author: Jeff Zhang <zj...@apache.org>
Authored: Fri Jun 1 21:58:40 2018 +0800
Committer: Jeff Zhang <zj...@apache.org>
Committed: Thu Jun 7 15:23:39 2018 +0800

----------------------------------------------------------------------
 .../src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zeppelin/blob/6fc3057d/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java
----------------------------------------------------------------------
diff --git a/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java b/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java
index 5deb52d..76e2103 100644
--- a/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java
+++ b/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java
@@ -495,7 +495,8 @@ public class LivyInterpreterIT {
       InterpreterResult result = sparkInterpreter.interpret("sc.version", context);
       assertEquals(InterpreterResult.Code.SUCCESS, result.code());
       assertEquals(2, result.message().size());
-      assertTrue(result.message().get(1).getData().contains("Spark Application Id"));
+      // check yarn appId and ensure it is not null
+      assertTrue(result.message().get(1).getData().contains("Spark Application Id: application_"));
 
       // html output
       String htmlCode = "println(\"%html <h1> hello </h1>\")";