Repository: zeppelin
Updated Branches:
  refs/heads/branch-0.8 81579da87 -> 32698dc19


[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 <zjf...@apache.org>

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

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

(cherry picked from commit 6fc3057d529691e697361ef9a67e21cac75e21ed)
Signed-off-by: Jeff Zhang <zjf...@apache.org>


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

Branch: refs/heads/branch-0.8
Commit: 32698dc19582a46926456d954c3df58b592f7f0e
Parents: 81579da
Author: Jeff Zhang <zjf...@apache.org>
Authored: Fri Jun 1 21:58:40 2018 +0800
Committer: Jeff Zhang <zjf...@apache.org>
Committed: Thu Jun 7 15:23:50 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/32698dc1/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 c7fbc8d..fd3c253 100644
--- a/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java
+++ b/livy/src/test/java/org/apache/zeppelin/livy/LivyInterpreterIT.java
@@ -491,7 +491,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>\")";

Reply via email to